Hi,
On Thu, Oct 6, 2011 at 17:31, Daniel K. <d...@uw.no> wrote:
> 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.
The new patch seems to be indeed better. My only concern is whether it
covers all those cases now. Did you try passing all possible kinds of
value expressions ? Or is it guaranteed to cover all cases by design?
If not, we should probably add more of those cases in the tests.
Other than that, this looks fine.
>
> 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.
>
--
Etienne Kneuss
http://www.colder.ch
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
