I want to propose the following patch to Zend to fix a minor
inconsistency. The sort() functions reindex arrays of length > 1, but
not of length == 1.

See http://bugs.php.net/bug.php?id=12572 for an example.

The patch is:

Index: zend_hash.c
===================================================================
RCS file: /repository/Zend/zend_hash.c,v
retrieving revision 1.86
diff -u -u -r1.86 zend_hash.c
--- zend_hash.c 1 Aug 2002 16:03:21 -0000       1.86
+++ zend_hash.c 17 Aug 2002 06:38:14 -0000
@@ -1115,7 +1115,7 @@
 
        IS_CONSISTENT(ht);
 
-       if (ht->nNumOfElements <= 1) {  /* Doesn't require sorting */
+       if (ht->nNumOfElements < 1) {   /* Doesn't require sorting */
                return SUCCESS;
        }
        arTmp = (Bucket **) pemalloc(ht->nNumOfElements * sizeof(Bucket *), 
ht->persistent);

If this seems good, can someone with Zend permission please apply?

Thanks.

-adam

-- 
adam maccabee trachtenberg
[EMAIL PROTECTED]


-- 
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to