Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:89962
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.220.43 as permitted sender)
To: bishop@php.net, PHP internals <internals@lists.php.net>
References: <CAEYWF=7Px5LWGdGqHdmVwt_tWd-XAUhZzUFoOa_rDmtpi88BAA@mail.gmail.com>
Message-ID: <568742A6.8070400@gmail.com>
Date: Fri, 1 Jan 2016 19:23:18 -0800
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:38.0)
 Gecko/20100101 Thunderbird/38.4.0
MIME-Version: 1.0
In-Reply-To: <CAEYWF=7Px5LWGdGqHdmVwt_tWd-XAUhZzUFoOa_rDmtpi88BAA@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] [RFC] On-Demand Name Mangling
From: smalyshev@gmail.com (Stanislav Malyshev)

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