Commit: f5bf1ba88fbacfebbb59b1e7bf43dd73649fa686 Author: Ben Ramsey <ram...@php.net> Tue, 19 Mar 2013 20:44:19 -0500 Parents: 3439a098a0b646ff05d4da9748996214cac39d12 Branches: PHP-5.5 master
Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=f5bf1ba88fbacfebbb59b1e7bf43dd73649fa686 Log: array_column: Fix compile-time warnings Changed paths: M ext/standard/array.c Diff: diff --git a/ext/standard/array.c b/ext/standard/array.c index fa19737..6983764 100644 --- a/ext/standard/array.c +++ b/ext/standard/array.c @@ -2571,9 +2571,9 @@ PHP_FUNCTION(array_column) zval *zarray, *zcolumn, *zkey = NULL, **data, **zcolval, **zkeyval; HashTable *arr_hash; HashPosition pointer; - ulong column_idx = 0, key_idx = 0, keyval_idx = 0; + ulong column_idx = 0, key_idx = 0; char *column = NULL, *key = NULL, *keyval = NULL; - int column_len = 0, key_len = 0; + int column_len = 0, key_len = 0, keyval_idx = -1; if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "az|z", &zarray, &zcolumn, &zkey) == FAILURE) { return; @@ -2640,13 +2640,13 @@ PHP_FUNCTION(array_column) Z_ADDREF_PP(zcolval); keyval = NULL; - keyval_idx = NULL; + keyval_idx = -1; if (zkey) { if (key && zend_hash_find(Z_ARRVAL_PP(data), key, key_len + 1, (void**)&zkeyval) == FAILURE) { - keyval_idx = NULL; + keyval_idx = -1; } else if (!key && zend_hash_index_find(Z_ARRVAL_PP(data), key_idx, (void**)&zkeyval) == FAILURE) { - keyval_idx = NULL; + keyval_idx = -1; } else { switch (Z_TYPE_PP(zkeyval)) { case IS_LONG: @@ -2660,14 +2660,14 @@ PHP_FUNCTION(array_column) keyval = Z_STRVAL_PP(zkeyval); break; default: - keyval_idx = NULL; + keyval_idx = -1; } } } if (keyval) { add_assoc_zval(return_value, keyval, *zcolval); - } else if (keyval_idx != NULL) { + } else if (keyval_idx != -1) { add_index_zval(return_value, keyval_idx, *zcolval); } else { add_next_index_zval(return_value, *zcolval); -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php