Fixed compilation warnings

diff --git a/Zend/zend_alloc.c b/Zend/zend_alloc.c
index 76f1952f93d81aa956dcbcee8d445001a976e135..ed7575d256391a23e723f7e2f4c0f8fea88702a7 100644 (file)
--- a/Zend/zend_alloc.c
+++ b/Zend/zend_alloc.c
@@ -2374,7 +2374,7 @@ ZEND_API void ZEND_FASTCALL _efree_large(void *ptr, size_t size)
                size_t page_offset = ZEND_MM_ALIGNED_OFFSET(ptr, ZEND_MM_CHUNK_SIZE);
                zend_mm_chunk *chunk = (zend_mm_chunk*)ZEND_MM_ALIGNED_BASE(ptr, ZEND_MM_CHUNK_SIZE);
                int page_num = page_offset / ZEND_MM_PAGE_SIZE;
-               int pages_count = ZEND_MM_ALIGNED_SIZE_EX(size, ZEND_MM_PAGE_SIZE) / ZEND_MM_PAGE_SIZE;
+               uint32_t pages_count = ZEND_MM_ALIGNED_SIZE_EX(size, ZEND_MM_PAGE_SIZE) / ZEND_MM_PAGE_SIZE;
 
                ZEND_MM_CHECK(chunk->heap == AG(mm_heap) && ZEND_MM_ALIGNED_OFFSET(page_offset, ZEND_MM_PAGE_SIZE) == 0, "zend_mm_heap corrupted");
                ZEND_ASSERT(chunk->map[page_num] & ZEND_MM_IS_LRUN);

diff --git a/Zend/zend_builtin_functions.c b/Zend/zend_builtin_functions.c
index b9d8341b523b1726e988618e3e6119eb13160d96..c84a1f1bcde0f75d9b447edf9a83f0a9f4e353d0 100644 (file)
--- a/Zend/zend_builtin_functions.c
+++ b/Zend/zend_builtin_functions.c
@@ -767,7 +767,7 @@ ZEND_FUNCTION(each)
    Return the current error_reporting level, and if an argument was passed - change to the new level */
 ZEND_FUNCTION(error_reporting)
 {
-       zval *err;
+       zval *err = NULL;
        int old_error_reporting;
 
 #ifndef FAST_ZPP

diff --git a/ext/date/php_date.c b/ext/date/php_date.c
index fd8f9f88380714a6ca99997e9646858d0d5433be..5e9c15c19df0341792b17d24982d56fd50843b11 100644 (file)
--- a/ext/date/php_date.c
+++ b/ext/date/php_date.c
@@ -3100,7 +3100,7 @@ PHP_METHOD(DateTimeImmutable, modify)
                RETURN_FALSE;
        }
 
-       ZVAL_COPY_VALUE(return_value, &new_object);
+       ZVAL_OBJ(return_value, Z_OBJ(new_object));
 }
 /* }}} */
 
@@ -3151,7 +3151,7 @@ PHP_METHOD(DateTimeImmutable, add)
        date_clone_immutable(object, &new_object);
        php_date_add(&new_object, interval, return_value);
 
-       ZVAL_COPY_VALUE(return_value, &new_object);
+       ZVAL_OBJ(return_value, Z_OBJ(new_object));
 }
 /* }}} */
 
@@ -3207,7 +3207,7 @@ PHP_METHOD(DateTimeImmutable, sub)
        date_clone_immutable(object, &new_object);
        php_date_sub(&new_object, interval, return_value);
 
-       ZVAL_COPY_VALUE(return_value, &new_object);
+       ZVAL_OBJ(return_value, Z_OBJ(new_object));
 }
 /* }}} */
 
@@ -3311,7 +3311,7 @@ PHP_METHOD(DateTimeImmutable, setTimezone)
        date_clone_immutable(object, &new_object);
        php_date_timezone_set(&new_object, timezone_object, return_value);
 
-       ZVAL_COPY_VALUE(return_value, &new_object);
+       ZVAL_OBJ(return_value, Z_OBJ(new_object));
 }
 /* }}} */
 
@@ -3395,7 +3395,7 @@ PHP_METHOD(DateTimeImmutable, setTime)
        date_clone_immutable(object, &new_object);
        php_date_time_set(&new_object, h, i, s, return_value);
 
-       ZVAL_COPY_VALUE(return_value, &new_object);
+       ZVAL_OBJ(return_value, Z_OBJ(new_object));
 }
 /* }}} */
 
@@ -3444,7 +3444,7 @@ PHP_METHOD(DateTimeImmutable, setDate)
        date_clone_immutable(object, &new_object);
        php_date_date_set(&new_object, y, m, d, return_value);
 
-       ZVAL_COPY_VALUE(return_value, &new_object);
+       ZVAL_OBJ(return_value, Z_OBJ(new_object));
 }
 /* }}} */
 
@@ -3497,7 +3497,7 @@ PHP_METHOD(DateTimeImmutable, setISODate)
        date_clone_immutable(object, &new_object);
        php_date_isodate_set(&new_object, y, w, d, return_value);
 
-       ZVAL_COPY_VALUE(return_value, &new_object);
+       ZVAL_OBJ(return_value, Z_OBJ(new_object));
 }
 /* }}} */
 
@@ -3544,7 +3544,7 @@ PHP_METHOD(DateTimeImmutable, setTimestamp)
        date_clone_immutable(object, &new_object);
        php_date_timestamp_set(&new_object, timestamp, return_value);
 
-       ZVAL_COPY_VALUE(return_value, &new_object);
+       ZVAL_OBJ(return_value, Z_OBJ(new_object));
 }
 /* }}} */
 

diff --git a/ext/dom/php_dom.c b/ext/dom/php_dom.c
index b6dc791d3d5c948c544fd67162bd045cb8abb4fb..68a8d3a802a43ac38fd5a0d84100cff7ea0cb347 100644 (file)
--- a/ext/dom/php_dom.c
+++ b/ext/dom/php_dom.c
@@ -420,7 +420,7 @@ static HashTable* dom_get_debug_info_helper(zval *object, int *is_temp) /* {{{ *
                                                *std_props;
        zend_string                     *string_key;
        dom_prop_handler        *entry;
-       zval object_value;
+       zend_string         *object_str;
 
        *is_temp = 1;
 
@@ -431,7 +431,7 @@ static HashTable* dom_get_debug_info_helper(zval *object, int *is_temp) /* {{{ *
                return debug_info;
        }
 
-       ZVAL_STRING(&object_value, "(object value omitted)");
+       object_str = zend_string_init("(object value omitted)", sizeof("(object value omitted)")-1, 0);
 
        ZEND_HASH_FOREACH_STR_KEY_PTR(prop_handlers, string_key, entry) {
                zval value;
@@ -442,13 +442,14 @@ static HashTable* dom_get_debug_info_helper(zval *object, int *is_temp) /* {{{ *
 
                if (Z_TYPE(value) == IS_OBJECT) {
                        zval_dtor(&value);
-                       ZVAL_COPY(&value, &object_value);
+                       ZVAL_NEW_STR(&value, object_str);
+                       zend_string_addref(object_str);
                }
 
                zend_hash_add(debug_info, string_key, &value);
        } ZEND_HASH_FOREACH_END();
 
-       zval_dtor(&object_value);
+       zend_string_release(object_str);
 
        return debug_info;
 }

diff --git a/ext/intl/transliterator/transliterator_class.c b/ext/intl/transliterator/transliterator_class.c
index 5c80b25af868a7414097de6682381469a4c7be1d..ba207a021cdb0e4c2388087fcbdfb98e88c5f42e 100644 (file)
--- a/ext/intl/transliterator/transliterator_class.c
+++ b/ext/intl/transliterator/transliterator_class.c
@@ -311,20 +311,11 @@ ZEND_BEGIN_ARG_INFO_EX( ainfo_trans_create_from_rules, 0, 0, 1 )
        ZEND_ARG_INFO( 0, direction )
 ZEND_END_ARG_INFO()
 
-ZEND_BEGIN_ARG_INFO_EX( ainfo_trans_create_inverse, 0, 0, 1 )
-       ZEND_ARG_OBJ_INFO( 0, orig_trans, Transliterator, 0 )
-ZEND_END_ARG_INFO()
-
 ZEND_BEGIN_ARG_INFO_EX( ainfo_trans_me_transliterate, 0, 0, 1 )
        ZEND_ARG_INFO( 0, subject )
        ZEND_ARG_INFO( 0, start )
        ZEND_ARG_INFO( 0, end )
 ZEND_END_ARG_INFO()
-
-ZEND_BEGIN_ARG_INFO_EX( ainfo_trans_error, 0, 0, 1 )
-       ZEND_ARG_OBJ_INFO( 0, trans, Transliterator, 0 )
-ZEND_END_ARG_INFO()
-
 /* }}} */
 
 /* {{{ Transliterator_class_functions

diff --git a/ext/mysqli/mysqli_fe.c b/ext/mysqli/mysqli_fe.c
index fe273ee6e2966d83f225e0e30f40167c4d60635b..00dc1ca01ac4facd23ee241b4fa8b7a74b1c1f92 100644 (file)
--- a/ext/mysqli/mysqli_fe.c
+++ b/ext/mysqli/mysqli_fe.c
@@ -63,10 +63,6 @@ ZEND_BEGIN_ARG_INFO(arginfo_class_mysqli_stmt_bind_param, 0)
        ZEND_ARG_VARIADIC_INFO(1, vars)
 ZEND_END_ARG_INFO()
 
-ZEND_BEGIN_ARG_INFO(all_args_force_by_ref, 0)
-       ZEND_ARG_VARIADIC_INFO(1, vars)
-ZEND_END_ARG_INFO()
-
 ZEND_BEGIN_ARG_INFO_EX(arginfo_mysqli_poll, 0, 0, 4)
        ZEND_ARG_ARRAY_INFO(1, read, 1)
        ZEND_ARG_ARRAY_INFO(1, write, 1)

diff --git a/ext/openssl/openssl.c b/ext/openssl/openssl.c
index 95e5b54f1e9ef43d9bb5e9b79d199d5281234241..bb22d9d4034dcb239ce3082116676d44710203dd 100644 (file)
--- a/ext/openssl/openssl.c
+++ b/ext/openssl/openssl.c
@@ -5733,7 +5733,7 @@ PHP_FUNCTION(openssl_get_curve_names)
 {
        EC_builtin_curve *curves = NULL;
        const char *sname;
-       int i;
+       size_t i;
        size_t len = EC_get_builtin_curves(NULL, 0);
 
        curves = emalloc(sizeof(EC_builtin_curve) * len);

diff --git a/ext/readline/readline.c b/ext/readline/readline.c
index 0e6f849444846eeda5f52dfbe27ab8d23023e217..9c637427f7540baefd75f21eec003d22ce3acedd 100644 (file)
--- a/ext/readline/readline.c
+++ b/ext/readline/readline.c
@@ -122,9 +122,11 @@ ZEND_END_ARG_INFO()
 ZEND_BEGIN_ARG_INFO(arginfo_readline_redisplay, 0)
 ZEND_END_ARG_INFO()
 
+#if HAVE_RL_ON_NEW_LINE
 ZEND_BEGIN_ARG_INFO(arginfo_readline_on_new_line, 0)
 ZEND_END_ARG_INFO()
 #endif
+#endif
 /* }}} */
 
 /* {{{ module stuff */

diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c
index 1eaa9d17b02fb9961758602f33fc2dc2817dc980..6471e4aa2aa85bfe9c9763c1f32eaa1f97b23dc7 100644 (file)
--- a/ext/reflection/php_reflection.c
+++ b/ext/reflection/php_reflection.c
@@ -1398,7 +1398,6 @@ static void reflection_class_constant_factory(zend_class_entry *ce, zend_string
 static void _reflection_export(INTERNAL_FUNCTION_PARAMETERS, zend_class_entry *ce_ptr, int ctor_argc)
 {
        zval reflector;
-       zval output, *output_ptr = &output;
        zval *argument_ptr, *argument2_ptr;
        zval retval, params[2];
        int result;
@@ -1455,9 +1454,8 @@ static void _reflection_export(INTERNAL_FUNCTION_PARAMETERS, zend_class_entry *c
        }
 
        /* Call static reflection::export */
-       ZVAL_BOOL(&output, return_output);
        ZVAL_COPY_VALUE(&params[0], &reflector);
-       ZVAL_COPY_VALUE(&params[1], output_ptr);
+       ZVAL_BOOL(&params[1], return_output);
 
        ZVAL_STRINGL(&fci.function_name, "reflection::export", sizeof("reflection::export") - 1);
        fci.object = NULL;

diff --git a/ext/spl/spl_directory.c b/ext/spl/spl_directory.c
index 094a4bc36761872078ebe06bc5fbfc38eee5a46a..2639065286a049bec616ea38fc82eb877ad0db18 100644 (file)
--- a/ext/spl/spl_directory.c
+++ b/ext/spl/spl_directory.c
@@ -1851,7 +1851,7 @@ static int spl_filesystem_object_cast(zval *readobj, zval *writeobj, int type)
 
                                ZVAL_STRINGL(retval_ptr, intern->file_name, intern->file_name_len);
                                zval_ptr_dtor(readobj);
-                               ZVAL_COPY_VALUE(writeobj, retval_ptr);
+                               ZVAL_NEW_STR(writeobj, Z_STR_P(retval_ptr));
                        } else {
                                ZVAL_STRINGL(writeobj, intern->file_name, intern->file_name_len);
                        }
@@ -1863,7 +1863,7 @@ static int spl_filesystem_object_cast(zval *readobj, zval *writeobj, int type)
 
                                ZVAL_STRING(retval_ptr, intern->u.dir.entry.d_name);
                                zval_ptr_dtor(readobj);
-                               ZVAL_COPY_VALUE(writeobj, retval_ptr);
+                               ZVAL_NEW_STR(writeobj, Z_STR_P(retval_ptr));
                        } else {
                                ZVAL_STRING(writeobj, intern->u.dir.entry.d_name);
                        }

diff --git a/ext/standard/basic_functions.c b/ext/standard/basic_functions.c
index 439c0fdbae1b904a72118d78f650f705c83a05a8..22bc77f77fe04e8db8f723ce0c172459070d1933 100644 (file)
--- a/ext/standard/basic_functions.c
+++ b/ext/standard/basic_functions.c
@@ -2676,6 +2676,7 @@ ZEND_BEGIN_ARG_INFO_EX(arginfo_version_compare, 0, 0, 2)
 ZEND_END_ARG_INFO()
 /* }}} */
 /* {{{ win32/codepage.c */
+#ifdef PHP_WIN32
 ZEND_BEGIN_ARG_INFO_EX(arginfo_sapi_windows_cp_set, 0, 0, 1)
        ZEND_ARG_TYPE_INFO(0, code_page, IS_LONG, 0)
 ZEND_END_ARG_INFO()
@@ -2691,6 +2692,7 @@ ZEND_BEGIN_ARG_INFO_EX(arginfo_sapi_windows_cp_conv, 0, 0, 3)
        ZEND_ARG_INFO(0, out_codepage)
        ZEND_ARG_TYPE_INFO(0, subject, IS_STRING, 0)
 ZEND_END_ARG_INFO()
+#endif
 /* }}} */
 /* }}} */
 

diff --git a/main/fastcgi.c b/main/fastcgi.c
index 838fe386d2800d8e9593dc1f6b935935927e813a..f02a168008fd0bd4e1ac7f8da2ec45a7dc6dbba4 100644 (file)
--- a/main/fastcgi.c
+++ b/main/fastcgi.c
@@ -1081,11 +1081,14 @@ static int fcgi_read_request(fcgi_request *req)
        req->id = (hdr.requestIdB1 << 8) + hdr.requestIdB0;
 
        if (hdr.type == FCGI_BEGIN_REQUEST && len == sizeof(fcgi_begin_request)) {
+               fcgi_begin_request *b;
+
                if (safe_read(req, buf, len+padding) != len+padding) {
                        return 0;
                }
 
-               req->keep = (((fcgi_begin_request*)buf)->flags & FCGI_KEEP_CONN);
+               b = (fcgi_begin_request*)buf;
+               req->keep = (b->flags & FCGI_KEEP_CONN);
 #ifdef TCP_NODELAY
                if (req->keep && req->tcp && !req->nodelay) {
 # ifdef _WIN32
@@ -1098,7 +1101,7 @@ static int fcgi_read_request(fcgi_request *req)
                        req->nodelay = 1;
                }
 #endif
-               switch ((((fcgi_begin_request*)buf)->roleB1 << 8) + ((fcgi_begin_request*)buf)->roleB0) {
+               switch ((b->roleB1 << 8) + b->roleB0) {
                        case FCGI_RESPONDER:
                                fcgi_hash_set(&req->env, FCGI_HASH_FUNC("FCGI_ROLE", sizeof("FCGI_ROLE")-1), "FCGI_ROLE", sizeof("FCGI_ROLE")-1, "RESPONDER", sizeof("RESPONDER")-1);
                                break;

diff --git a/sapi/cgi/cgi_main.c b/sapi/cgi/cgi_main.c
index 8a263e59f6021075cef6d79cb75e09b44dde061d..b4ba497e6714ecacbc816b68d424b68ca3281beb 100644 (file)
--- a/sapi/cgi/cgi_main.c
+++ b/sapi/cgi/cgi_main.c
@@ -276,8 +276,9 @@ static int print_extension_info(zend_extension *ext, void *arg)
 
 static int extension_name_cmp(const zend_llist_element **f, const zend_llist_element **s)
 {
-       return strcmp(  ((zend_extension *)(*f)->data)->name,
-                                       ((zend_extension *)(*s)->data)->name);
+       zend_extension *fe = (zend_extension*)(*f)->data;
+       zend_extension *se = (zend_extension*)(*s)->data;
+       return strcmp(fe->name, se->name);
 }
 
 static void print_extensions(void)

diff --git a/sapi/cli/php_cli.c b/sapi/cli/php_cli.c
index 9c80ffd4adbdb91b418504849c554698eca8e118..594f3ff611a36e5fb8c8b5f24f5c15e356f90e10 100644 (file)
--- a/sapi/cli/php_cli.c
+++ b/sapi/cli/php_cli.c
@@ -222,8 +222,9 @@ static int print_extension_info(zend_extension *ext, void *arg) /* {{{ */
 
 static int extension_name_cmp(const zend_llist_element **f, const zend_llist_element **s) /* {{{ */
 {
-       return strcmp(((zend_extension *)(*f)->data)->name,
-                                 ((zend_extension *)(*s)->data)->name);
+       zend_extension *fe = (zend_extension*)(*f)->data;
+       zend_extension *se = (zend_extension*)(*s)->data;
+       return strcmp(fe->name, se->name);
 }
 /* }}} */
 
@@ -557,7 +558,6 @@ static php_stream *s_in_process = NULL;
 
 static void cli_register_file_handles(void) /* {{{ */
 {
-       zval zin, zout, zerr;
        php_stream *s_in, *s_out, *s_err;
        php_stream_context *sc_in=NULL, *sc_out=NULL, *sc_err=NULL;
        zend_constant ic, oc, ec;
@@ -581,23 +581,20 @@ static void cli_register_file_handles(void) /* {{{ */
 
        s_in_process = s_in;
 
-       php_stream_to_zval(s_in,  &zin);
-       php_stream_to_zval(s_out, &zout);
-       php_stream_to_zval(s_err, &zerr);
+       php_stream_to_zval(s_in,  &ic.value);
+       php_stream_to_zval(s_out, &oc.value);
+       php_stream_to_zval(s_err, &ec.value);
 
-       ZVAL_COPY_VALUE(&ic.value, &zin);
        ic.flags = CONST_CS;
        ic.name = zend_string_init("STDIN", sizeof("STDIN")-1, 1);
        ic.module_number = 0;
        zend_register_constant(&ic);
 
-       ZVAL_COPY_VALUE(&oc.value, &zout);
        oc.flags = CONST_CS;
        oc.name = zend_string_init("STDOUT", sizeof("STDOUT")-1, 1);
        oc.module_number = 0;
        zend_register_constant(&oc);
 
-       ZVAL_COPY_VALUE(&ec.value, &zerr);
        ec.flags = CONST_CS;
        ec.name = zend_string_init("STDERR", sizeof("STDERR")-1, 1);
        ec.module_number = 0;

diff --git a/sapi/fpm/fpm/fpm_main.c b/sapi/fpm/fpm/fpm_main.c
index a3b0cc9840132cc07b366ebb17c0589b657b10d4..e3786070c73ed0f7775d02a579ca252a1898c2f1 100644 (file)
--- a/sapi/fpm/fpm/fpm_main.c
+++ b/sapi/fpm/fpm/fpm_main.c
@@ -258,8 +258,9 @@ static int print_extension_info(zend_extension *ext, void *arg) /* {{{ */
 
 static int extension_name_cmp(const zend_llist_element **f, const zend_llist_element **s) /* {{{ */
 {
-       return strcmp(  ((zend_extension *)(*f)->data)->name,
-                                       ((zend_extension *)(*s)->data)->name);
+       zend_extension *fe = (zend_extension*)(*f)->data;
+       zend_extension *se = (zend_extension*)(*s)->data;
+       return strcmp(fe->name, se->name);
 }
 /* }}} */