Re: foreach with null
| From: | Ants Aasma | Date: | Tue, 25 Oct 2005 15:39:51 +0000 |
| Subject: | Re: foreach with null | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-19717@lists.php.net to get a copy of this message | ||
It currently is equal:
$ php -r 'var_dump(null == array()); var_dump((array) null);'
bool(true)
array(0) {
}
Ants
Jani Taskinen wrote:
Will the patch make it silently ignore also empty string and 0? If not, -1. (what's with null being suddenly equal to empty array? :) --Jani On Tue, 25 Oct 2005, Marcus Boerger wrote:Hello internals, i would like to change foreach a tiny bit. Actually i don't like the warning when using foreach with null. marcus@zaphod /usr/src/php-cvs $ php -r '$n=NULL; foreach($n as $v);' make: `sapi/cli/php' is up to date. Warning: Invalid argument supplied for foreach() in Command line code on line 1 marcus@zaphod /usr/src/php-cvs $ php -r 'foreach(NULL as $v);' make: `sapi/cli/php' is up to date. Warning: Invalid argument supplied for foreach() in Command line code on line 1 The way we use NULL in PHP should result in no warning here. The patch for this can be found here: http://php.net/~helly/php/ext/ze2/ze2-foreach-null-20051025.diff.txt It was made with head but applies to 5.1 as well. Any opinions? Best regards, Marcus