felipe          Tue Feb 12 01:49:43 2008 UTC

  Modified files:              
    /php-src/ext/standard       array.c 
  Log:
  MFB: array_slice() - Fixed behavior when NULL is given in third parameter (BC)
  
http://cvs.php.net/viewvc.cgi/php-src/ext/standard/array.c?r1=1.442&r2=1.443&diff_format=u
Index: php-src/ext/standard/array.c
diff -u php-src/ext/standard/array.c:1.442 php-src/ext/standard/array.c:1.443
--- php-src/ext/standard/array.c:1.442  Mon Feb 11 13:34:26 2008
+++ php-src/ext/standard/array.c        Tue Feb 12 01:49:42 2008
@@ -21,7 +21,7 @@
    +----------------------------------------------------------------------+
 */
 
-/* $Id: array.c,v 1.442 2008/02/11 13:34:26 tony2001 Exp $ */
+/* $Id: array.c,v 1.443 2008/02/12 01:49:42 felipe Exp $ */
 
 #include "php.h"
 #include "php_ini.h"
@@ -2294,6 +2294,7 @@
 PHP_FUNCTION(array_slice)
 {
        zval     *input,                /* Input array */
+                       **z_length,             /* How many elements to get */ 
                        **entry;                /* An array entry */
        long     offset,                /* Offset to get elements from */
                         length = 0;
@@ -2305,7 +2306,7 @@
        ulong num_key;
        HashPosition hpos;
 
-       if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "al|lb", &input, 
&offset, &length, &preserve_keys) == FAILURE) {
+       if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "al|Zb", &input, 
&offset, &z_length, &preserve_keys) == FAILURE) {
                return;
        }
 
@@ -2313,8 +2314,11 @@
        num_in = zend_hash_num_elements(Z_ARRVAL_P(input));
 
        /* We want all entries from offset to the end if length is not passed 
or length is null */
-       if (length == 0) {
+       if (ZEND_NUM_ARGS() < 3 || Z_TYPE_PP(z_length) == IS_NULL) {
                length = num_in;
+       } else {
+               convert_to_long_ex(z_length);
+               length = Z_LVAL_PP(z_length);
        }
 
        /* Initialize returned array */

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to