Re: [RFC] On-Demand Name Mangling
| From: | Stanislav Malyshev | Date: | Sat, 02 Jan 2016 03:23:18 +0000 |
| Subject: | Re: [RFC] On-Demand Name Mangling | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-89962@lists.php.net to get a copy of this message | ||
Hi!
> Ringing in the new year with a proposal to retool name mangling:
While the variable name changing probably outlived its usefulness with
the demise of register_globals, changing it would produce serious BC
issues with which we should be very careful. Emitting E_DEPRECATED based
on user data certainly does not sound like a good idea.
I like Nikita's proposal on both counts - making it an config switch (as
much as I hate those) and have extract() flag for this (I would make the
current behavior default to minimize BC disruption).
I don't see why it says in the RFC it should be PHP_INI_ALL - I think it
should be PHP_INI_SYSTEM or PHP_INI_PERDIR, as changing it at runtime
won't do anything useful - the values should already be loaded.
I would propose default for the INI to be current behavior for now and 0
for next version where we can break BC.
BTW, for extract() option we don't need 2/3 majority, it's a simple
function fix (which can be merged even in 7.0).
--
Stas Malyshev
smalyshev@gmail.com