Folks,
I'd say throw the warning if the switch is on and someone does
call-time-pass-by-reference and turn that switch off by default.
That's exactly the situation we have now - a warning when the INI setting
is changed, for code that by default runs silently.
And we offer a recommendation to throw that warning in production. That's
the real WTF!
If call-time pass-by-ref is something that needs to be warned about, warn
by default so everyone will get to see it during development. If it isn't
something that needs to be warned about, why throw a warning at all?
Did I hit the wrong season with this? Could someone with karma and history
please respond?
On investigation, it seems allow_call_time_pass_reference shouldn't be
E_DEPRECATED but E_STRICT, assuming it should throw a warning at all. It
hasn't been removed in CVS HEAD, there are occasional genuine uses for it,
and going by Stas' oblique message earlier in the week there may well be
future genuine uses for it too, even if Derick mutters sth off-list about
the spawn of Satan.
Off by default, off in php.ini-dist, possibly on in php.ini-recommended for
BC reasons?
It's still the wrong way around.
- Steph
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
