Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:98406
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.128.175 as permitted sender)
To: PHP Internals <internals@lists.php.net>
References: <0c5fc86b-5050-fe73-7afa-b3e5a38ad370@gmail.com>
Message-ID: <db9b4f41-c032-38e6-ca67-352d22364b57@gmail.com>
Date: Sun, 5 Mar 2017 22:45:12 +0000
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101
 Thunderbird/45.7.1
MIME-Version: 1.0
In-Reply-To: <0c5fc86b-5050-fe73-7afa-b3e5a38ad370@gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [RFC] Deprecate and Remove Bareword (Unquoted) Strings
From: rowan.collins@gmail.com (Rowan Collins)

On 29/01/2017 19:33, Rowan Collins wrote:
> Currently, if the constant FROB_ACTIVE is not defined, the code "echo 
> FROB_ACTIVE;" results in an E_NOTICE and the string 'FROB_ACTIVE' 
> being displayed. I would like to propose that this be changed to an 
> E_WARNING in PHP 7.2, and to an Error in PHP 8.0.
> [...]
> https://wiki.php.net/rfc/deprecate-bareword-strings

Hi,

I have made some minor updates to the above RFC, and would like to open 
it for voting in the next few days.

Changes since I first posted the RFC are:

- Expanded the rationale for using E_WARNING rather than E_DEPRECATED. I 
always anticipated this would be the most contentious part of the 
proposal, but am convinced the peculiarities of the situation make this 
the best choice. Please read that section carefully if you are 
concerned, and let me know if anything remains unclear.

- Settled on a proposed wording for the new Warning. I spent some time 
considering the best phrasing and punctuation, but if anyone would like 
to suggest an improvement, I am open to feedback.

- Clarified the history of the feature, having successfully built PHP 2 
from the archived source, and confirmed that bare words were a syntax 
error. As such, no non-beta version of PHP allowed unquoted strings 
without issuing a Notice.

- Added links to Pull Requests for both php-src and php-langspec to 
implement the proposed wording. Amusingly, I had to change a test in 
php-src which seems to have been *accidentally* using an unquoted 
string; apparently, the configuration of that test is such that the 
E_NOTICE was suppressed.

Please let me know if there is any feedback I should take into 
consideration before opening this to a vote.

Regards,

-- 
Rowan Collins
[IMSoP]