]> granicus.if.org Git - php/commitdiff
Various warning fixes
authorNikita Popov <nikic@php.net>
Fri, 17 Jul 2015 18:18:25 +0000 (20:18 +0200)
committerNikita Popov <nikic@php.net>
Fri, 17 Jul 2015 18:18:25 +0000 (20:18 +0200)
ext/dom/documenttype.c
ext/gd/gd.c
ext/intl/collator/collator_sort.c
ext/intl/msgformat/msgformat_format.c
ext/intl/msgformat/msgformat_parse.c
ext/mbstring/php_mbregex.c
ext/pdo_mysql/php_pdo_mysql_int.h
ext/pdo_sqlite/sqlite_statement.c
ext/pgsql/pgsql.c
ext/readline/readline_cli.c

index c65902e759da02bea0202a97d0e7e5ae283c682d..30fd590536335c5e52a5d54a732f5f8ded855f80 100644 (file)
@@ -193,7 +193,7 @@ int dom_documenttype_internal_subset_read(dom_object *obj, zval *retval)
 #ifdef LIBXML2_NEW_BUFFER
                                smart_str_appendl(&ret_buf, (const char *) xmlOutputBufferGetContent(buff), xmlOutputBufferGetSize(buff));
 #else
-                               smart_str_appendl(&ret_buf, buff->buffer->content, buff->buffer->use);
+                               smart_str_appendl(&ret_buf, (char *) buff->buffer->content, buff->buffer->use);
 #endif
 
                                (void)xmlOutputBufferClose(buff);
index abd7b75c09616810572f71ba2ca676f592dc869e..7eaa9552d4c29f430f7ec8f16274c3e386afe639 100644 (file)
@@ -3857,10 +3857,9 @@ static void php_imagettftext_common(INTERNAL_FUNCTION_PARAMETERS, int mode, int
        if (extended && EXT) {  /* parse extended info */
                zval *item;
                zend_string *key;
-               zend_ulong num_key;
 
                /* walk the assoc array */
-               ZEND_HASH_FOREACH_KEY_VAL(HASH_OF(EXT), num_key, key, item) {
+               ZEND_HASH_FOREACH_STR_KEY_VAL(HASH_OF(EXT), key, item) {
                        if (key == NULL) {
                                continue;
                        }
index 8727c1d8ab8c3b1feddb6afe589d97334e026a2a..ca4b7ea785c753f34ea36ca200c0dec35466178f 100644 (file)
@@ -385,8 +385,6 @@ PHP_FUNCTION( collator_sort_with_sort_keys )
        int         utf16_buf_size       = DEF_UTF16_BUF_SIZE;       /* the length of utf16_buf */
        int         utf16_len            = 0;                        /* length of converted string */
 
-       HashTable* sortedHash            = NULL;
-
        COLLATOR_METHOD_INIT_VARS
 
        /* Parse parameters. */
index 5aa68e27354d85f49e037b0e6a842e692d52d265..959a4fd5a01ea1e917cf3c5915b7b7fdac5bd4c4 100644 (file)
@@ -104,7 +104,7 @@ PHP_FUNCTION( msgfmt_format_message )
        size_t      pattern_len = 0;
        const char *slocale = NULL;
        size_t      slocale_len = 0;
-       MessageFormatter_object mf = {0};
+       MessageFormatter_object mf;
        MessageFormatter_object *mfo = &mf;
 
        /* Parse parameters. */
@@ -117,6 +117,7 @@ PHP_FUNCTION( msgfmt_format_message )
                RETURN_FALSE;
        }
 
+       memset(mfo, 0, sizeof(*mfo));
        msgformat_data_init(&mfo->mf_data);
 
        if(pattern && pattern_len) {
index 4d63a0169a940e80bfa90289a1ea16d731eca5cf..349633912b73c0a346f97a319ddd0f18e23bffee 100644 (file)
@@ -97,7 +97,7 @@ PHP_FUNCTION( msgfmt_parse_message )
        size_t      slocale_len = 0;
        char       *source = NULL;
        size_t      src_len = 0;
-       MessageFormatter_object mf = {0};
+       MessageFormatter_object mf;
        MessageFormatter_object *mfo = &mf;
 
        /* Parse parameters. */
@@ -110,6 +110,7 @@ PHP_FUNCTION( msgfmt_parse_message )
                RETURN_FALSE;
        }
 
+       memset(mfo, 0, sizeof(*mfo));
        msgformat_data_init(&mfo->mf_data);
 
        if(pattern && pattern_len) {
index 88f8772d342341a71d9209d4e301a7eced127174..35a986ac2adb6d28b9a1e4b1838e0e3775c9b5a7 100644 (file)
@@ -909,7 +909,7 @@ static void _php_mb_regex_ereg_replace_exec(INTERNAL_FUNCTION_PARAMETERS, OnigOp
                        }
 #endif
                        /* copy the part of the string before the match */
-                       smart_str_appendl(&out_buf, pos, (size_t)((OnigUChar *)(string + regs->beg[0]) - pos));
+                       smart_str_appendl(&out_buf, (char *)pos, (size_t)((OnigUChar *)(string + regs->beg[0]) - pos));
 
                        if (!is_callable) {
                                /* copy replacement and backrefs */
@@ -992,14 +992,14 @@ static void _php_mb_regex_ereg_replace_exec(INTERNAL_FUNCTION_PARAMETERS, OnigOp
                                pos = (OnigUChar *)string + n;
                        } else {
                                if (pos < string_lim) {
-                                       smart_str_appendl(&out_buf, pos, 1);
+                                       smart_str_appendl(&out_buf, (char *)pos, 1);
                                }
                                pos++;
                        }
                } else { /* nomatch */
                        /* stick that last bit of string on our output */
                        if (string_lim - pos > 0) {
-                               smart_str_appendl(&out_buf, pos, string_lim - pos);
+                               smart_str_appendl(&out_buf, (char *)pos, string_lim - pos);
                        }
                }
                onig_region_free(regs, 0);
index 17bace7882eaa55e631bab4a8177c1593971fb2f..3d4108dd598e078662749efcfe54469ead68203e 100644 (file)
 #define PDO_DBG_ERR(msg) do { if (dbg_skip_trace == FALSE) PDO_MYSQL_G(dbg)->m->log(PDO_MYSQL_G(dbg), __LINE__, __FILE__, -1, "error: ", (msg)); } while (0)
 #define PDO_DBG_INF_FMT(...) do { if (dbg_skip_trace == FALSE) PDO_MYSQL_G(dbg)->m->log_va(PDO_MYSQL_G(dbg), __LINE__, __FILE__, -1, "info : ", __VA_ARGS__); } while (0)
 #define PDO_DBG_ERR_FMT(...) do { if (dbg_skip_trace == FALSE) PDO_MYSQL_G(dbg)->m->log_va(PDO_MYSQL_G(dbg), __LINE__, __FILE__, -1, "error: ", __VA_ARGS__); } while (0)
-#define PDO_DBG_ENTER(func_name) zend_bool dbg_skip_trace = TRUE; if (PDO_MYSQL_G(dbg)) dbg_skip_trace = !PDO_MYSQL_G(dbg)->m->func_enter(PDO_MYSQL_G(dbg), __LINE__, __FILE__, func_name, strlen(func_name));
+#define PDO_DBG_ENTER(func_name) \
+       zend_bool dbg_skip_trace = TRUE; \
+       ((void) dbg_skip_trace); \
+       if (PDO_MYSQL_G(dbg)) \
+               dbg_skip_trace = !PDO_MYSQL_G(dbg)->m->func_enter(PDO_MYSQL_G(dbg), __LINE__, __FILE__, func_name, strlen(func_name));
+
 #define PDO_DBG_RETURN(value)  do { if (PDO_MYSQL_G(dbg)) PDO_MYSQL_G(dbg)->m->func_leave(PDO_MYSQL_G(dbg), __LINE__, __FILE__, 0); return (value); } while (0)
 #define PDO_DBG_VOID_RETURN            do { if (PDO_MYSQL_G(dbg)) PDO_MYSQL_G(dbg)->m->func_leave(PDO_MYSQL_G(dbg), __LINE__, __FILE__, 0); return; } while (0)
 
index 53d12a719da3432c50a0714d02d04a36d5ca1573..9b74b24212a81754777bd98903af5cc4148c802e 100644 (file)
@@ -233,7 +233,7 @@ static int pdo_sqlite_stmt_fetch(pdo_stmt_t *stmt,
 static int pdo_sqlite_stmt_describe(pdo_stmt_t *stmt, int colno)
 {
        pdo_sqlite_stmt *S = (pdo_sqlite_stmt*)stmt->driver_data;
-       char *str;
+       const char *str;
 
        if(colno >= sqlite3_column_count(S->stmt)) {
                /* error invalid column */
index 9282d483645f3d7f87825baaabc6230d28d274ee..1172682bbc256a951bf6563af59eb3ca4cb4165b 100644 (file)
@@ -5803,7 +5803,6 @@ static int php_pgsql_add_quotes(zval *src, zend_bool should_free)
 PHP_PGSQL_API int php_pgsql_convert(PGconn *pg_link, const char *table_name, const zval *values, zval *result, zend_ulong opt)
 {
        zend_string *field = NULL;
-       zend_ulong num_idx = -1;
        zval meta, *def, *type, *not_null, *has_default, *is_enum, *val, new_val;
        int err = 0, skip_field;
        php_pgsql_data_type data_type;
@@ -5824,7 +5823,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn *pg_link, const char *table_name, con
                return FAILURE;
        }
 
-       ZEND_HASH_FOREACH_KEY_VAL(Z_ARRVAL_P(values), num_idx, field, val) {
+       ZEND_HASH_FOREACH_STR_KEY_VAL(Z_ARRVAL_P(values), field, val) {
                skip_field = 0;
                ZVAL_NULL(&new_val);
 
@@ -6550,7 +6549,6 @@ PHP_PGSQL_API int php_pgsql_insert(PGconn *pg_link, const char *table, zval *var
        char *tmp;
        smart_str querystr = {0};
        int ret = FAILURE;
-       zend_ulong num_idx;
        zend_string *fld;
 
        assert(pg_link != NULL);
@@ -6579,7 +6577,7 @@ PHP_PGSQL_API int php_pgsql_insert(PGconn *pg_link, const char *table, zval *var
        build_tablename(&querystr, pg_link, table);
        smart_str_appends(&querystr, " (");
 
-       ZEND_HASH_FOREACH_KEY(Z_ARRVAL_P(var_array), num_idx, fld) {
+       ZEND_HASH_FOREACH_STR_KEY(Z_ARRVAL_P(var_array), fld) {
                if (fld == NULL) {
                        php_error_docref(NULL, E_NOTICE, "Expects associative array for values to be inserted");
                        goto cleanup;
@@ -6748,11 +6746,10 @@ static inline int build_assignment_string(PGconn *pg_link, smart_str *querystr,
 {
        char *tmp;
        char buf[256];
-       zend_ulong num_idx;
        zend_string *fld;
        zval *val;
 
-       ZEND_HASH_FOREACH_KEY_VAL(ht, num_idx, fld, val) {
+       ZEND_HASH_FOREACH_STR_KEY_VAL(ht, fld, val) {
                if (fld == NULL) {
                        php_error_docref(NULL, E_NOTICE, "Expects associative array for values to be inserted");
                        return -1;
index 125b09e920f9028caf054df1986b8fe2c7862212..eaec591d9ae3b08b357006e9e8e8702b5f8ef47f 100644 (file)
@@ -587,7 +587,7 @@ static int readline_shell_run(void) /* {{{ */
 
        if (PG(auto_prepend_file) && PG(auto_prepend_file)[0]) {
                zend_file_handle *prepend_file_p;
-               zend_file_handle prepend_file = {0};
+               zend_file_handle prepend_file = {{0}};
 
                prepend_file.filename = PG(auto_prepend_file);
                prepend_file.opened_path = NULL;