]> granicus.if.org Git - php/commitdiff
Get rid of casts
authorAnatol Belski <ab@php.net>
Thu, 1 Mar 2018 19:52:09 +0000 (20:52 +0100)
committerAnatol Belski <ab@php.net>
Thu, 1 Mar 2018 19:52:09 +0000 (20:52 +0100)
ext/standard/basic_functions.c
ext/standard/basic_functions.h

index 398d44cf967f77b4a45ad75ec4a1c437e22a96a3..a0ad3b361982f5ffa1c020c38e68b81f7f800918 100644 (file)
@@ -4183,10 +4183,10 @@ PHP_FUNCTION(putenv)
 #endif
        }
 
-       pe.key_len = (int)strlen(pe.key);
+       pe.key_len = strlen(pe.key);
 #ifdef PHP_WIN32
        if (equals) {
-               if ((size_t)pe.key_len < setting_len - 1) {
+               if (pe.key_len < setting_len - 1) {
                        value = p + 1;
                } else {
                        /* empty string*/
@@ -4353,13 +4353,13 @@ PHP_FUNCTION(getopt)
        char *options = NULL, **argv = NULL;
        char opt[2] = { '\0' };
        char *optname;
-       int argc = 0, len, o;
-       size_t options_len = 0;
+       int argc = 0, o;
+       size_t options_len = 0, len;
        char *php_optarg = NULL;
        int php_optind = 1;
        zval val, *args = NULL, *p_longopts = NULL;
        zval *zoptind = NULL;
-       int optname_len = 0;
+       size_t optname_len = 0;
        opt_struct *opts, *orig_opts;
 
        ZEND_PARSE_PARAMETERS_START(1, 3)
@@ -4435,7 +4435,7 @@ PHP_FUNCTION(getopt)
 
                        opts->need_param = 0;
                        opts->opt_name = estrdup(ZSTR_VAL(arg_str));
-                       len = (int)strlen(opts->opt_name);
+                       len = strlen(opts->opt_name);
                        if ((len > 0) && (opts->opt_name[len - 1] == ':')) {
                                opts->need_param++;
                                opts->opt_name[len - 1] = '\0';
@@ -4491,7 +4491,7 @@ PHP_FUNCTION(getopt)
                }
 
                /* Add this option / argument pair to the result hash. */
-               optname_len = (int)strlen(optname);
+               optname_len = strlen(optname);
                if (!(optname_len > 1 && optname[0] == '0') && is_numeric_string(optname, optname_len, NULL, NULL, 0) == IS_LONG) {
                        /* numeric string */
                        int optname_int = atoi(optname);
index 7a7252b8e2757f6a9e780e192e19fa85c37052e8..073aa017ef05f0b14728182014bcfac9e9ea5331 100644 (file)
@@ -250,7 +250,7 @@ typedef struct {
        char *putenv_string;
        char *previous_value;
        char *key;
-       int key_len;
+       size_t key_len;
 } putenv_entry;
 #endif