Re: [PATCH] Fix for bug #55754

From: Date: Thu, 06 Oct 2011 15:31:00 +0000
Subject: Re: [PATCH] Fix for bug #55754
References: 1  Groups: php.internals 
Request: Send a blank email to internals+get-55706@lists.php.net to get a copy of this message
On 09/23/2011 02:06 PM, I wrote: > When a built-in function is defined with ZEND_SEND_PREFER_REF, PHP will > issue a strict warning if you use an assignment expression as the parameter. > > As an example, current() show this behaviour. > > <?php > current($foo = array("bar")); > ?> > > Presents you with: > > PHP Strict Standards: Only variables should be passed by reference in > %s on line %d A patch was appended, discussed, and improved, and I have uploaded a test-case, as well as a minimal patch that fixes the problem (attached) to the original bug-report. https://bugs.php.net/bug.php?id=55754 The patch still applies to trunk, and I think it should be applied to the 5.3 and 5.4 branches as well. What to do next? The feedback has been constructive, but I feel it has stagnated a bit. Of course, you may all be busy with other interesting stuff, but is there anything else you require from me to get this rolling? Should I explain better why I think it's an issue, and why the patch is needed? An indication as to if anyone is reviewing the proposed patch, and considering applying it, or telling me that this is the completely wrong approach to solve the problem and dropping a few hints would be most appreciated, but any feedback is welcome. Daniel K.

Attachment: [text/x-patch] bug55754_minimal.patch
« previous php.internals (#55706) next »