Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:19717 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 20141 invoked by uid 1010); 25 Oct 2005 15:39:50 -0000 Delivered-To: ezmlm-scan-internals@lists.php.net Delivered-To: ezmlm-internals@lists.php.net Received: (qmail 20126 invoked from network); 25 Oct 2005 15:39:50 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 25 Oct 2005 15:39:50 -0000 X-Host-Fingerprint: 195.80.104.106 gw.mig.ee Received: from ([195.80.104.106:21781] helo=localhost.localdomain) by pb1.pair.com (ecelerity 2.0 beta r(6323M)) with SMTP id B9/44-22886-5C15E534 for ; Tue, 25 Oct 2005 11:39:49 -0400 Message-ID: To: internals@lists.php.net Date: Tue, 25 Oct 2005 18:39:51 +0300 User-Agent: Mozilla Thunderbird 1.0.6 (X11/20050809) X-Accept-Language: en-us, en MIME-Version: 1.0 References: <425333877.20051025002259@marcus-boerger.de> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Posted-By: 195.80.104.106 Subject: Re: [PHP-DEV] foreach with null From: ants.aasma@mig.ee (Ants Aasma) 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 >> >> >