Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:98026
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain kelunik.com from 81.169.146.160 cause and error)
MIME-Version: 1.0
In-Reply-To: <0c5fc86b-5050-fe73-7afa-b3e5a38ad370@gmail.com>
References: <0c5fc86b-5050-fe73-7afa-b3e5a38ad370@gmail.com>
Date: Sun, 29 Jan 2017 21:42:55 +0100
Message-ID: <CANUQDChQc=iJ31XnFLSytoaG-F4juUGCEB0t2eycJo1dKUZf8A@mail.gmail.com>
To: Rowan Collins <rowan.collins@gmail.com>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=94eb2c065d985c8204054741be33
Subject: Re: [PHP-DEV] [RFC] Deprecate and Remove Bareword (Unquoted) Strings
From: me@kelunik.com (Niklas Keller)

--94eb2c065d985c8204054741be33
Content-Type: text/plain; charset=UTF-8

2017-01-29 20:33 GMT+01:00 Rowan Collins <rowan.collins@gmail.com>:

> Good evening all!
>
> 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.
>
> My reasoning for this includes:
>
> - consistency with other "undefined" errors, including the otherwise
> identical "echo \FROB_ACTIVE;"
> - the fallback to string appears to be largely undocumented except to
> discourage its use
> - the error can mask serious bugs, such as when mis-typing keywords
>
> I have tried to anticipate some of the questions that will be raised, so
> have drafted an RFC rather than trying to fit all the detail into one
> e-mail: https://wiki.php.net/rfc/deprecate-bareword-strings
>
> Please read my proposal, and let me know your thoughts. I have placed the
> RFC "under discussion", but will be happy to modify it based on feedback,
> and am in no haste to put it to a vote.
>
> Regards,
>
> --
> Rowan Collins
> [IMSoP]
>
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
I think it should emit an E_DEPRECATED instead of E_WARNING, but otherwise
+1.

--94eb2c065d985c8204054741be33--