Re: [PATCH] Fix for bug #55754
| From: | Daniel K. | 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
Attachment: [text/x-patch] bug55754_minimal.patch