Hannes,
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?
- Steph
-Hannes -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
-- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php