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 smalys...@gmail.com -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
