Re: cvs: php4 /ext/standard array.c
| From: | Jeroen van Wolffelaar | Date: | Fri, 05 Oct 2001 22:19:08 +0000 |
| Subject: | Re: cvs: php4 /ext/standard array.c | ||
| References: | 1 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-7689@lists.php.net to get a copy of this message | ||
Also, zend_hash_get_current_key_ex returns the strlen+1 as 5th
parameter. I would help a lot if the synopsis of zend-functions was
added just above the the function itself, together with things like it
duplicates or not. (though not relevant for
zend_hash_get_current_key_ex, but it is relevant in a lot of other
cases)
Open souce is good, but it shouldn't be source alone. The documentation
doesn't go into details like this.
Actually there's a comment that zend_hash_get_current_key_ex isn't
binary safe... but that's all.
Anyway, tests are very important for functions like in_array and
array_search, it's a kind of documentation-by-example. (which can be
checked more easily than you can check .xml documentation...)
--Jeroen