diff options
| author | Ben Ramsey <ramsey@php.net> | 2013-03-19 20:44:19 -0500 |
|---|---|---|
| committer | Ben Ramsey <ramsey@php.net> | 2013-03-19 20:44:19 -0500 |
| commit | f5bf1ba88fbacfebbb59b1e7bf43dd73649fa686 (patch) | |
| tree | 8327de854d9ee69dcf4d02a3593e30766357d077 /ext/standard/array.c | |
| parent | 3439a098a0b646ff05d4da9748996214cac39d12 (diff) | |
| download | php-git-f5bf1ba88fbacfebbb59b1e7bf43dd73649fa686.tar.gz | |
array_column: Fix compile-time warnings
Diffstat (limited to 'ext/standard/array.c')
| -rw-r--r-- | ext/standard/array.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/ext/standard/array.c b/ext/standard/array.c index fa19737522..6983764862 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); |