Hi Sara,
On Tue, Apr 19, 2016 at 8:24 AM, Sara Golemon <[email protected]> wrote:
> On Tue, Apr 12, 2016 at 7:38 PM, Sara Golemon <[email protected]> wrote:
>> https://wiki.php.net/rfc/octal.overload-checking
>> Because having this expression evaluate to true makes me sad: ("\000"
>> === "\400")
>>
> I haven't heard any responses on this and wanted to bump it back to
> the top of your newsreaders.
>
> For the record, my inclination is to raise a warning on octal
> overflow, but not change behavior (at least, not for 7.1, perhaps
> later).
>
> Note that this warning would bubble up during the lexing phase, so an
> opcache would prevent it from raising on every page load. That's
> probably fine, the alternative is to signal the parser to bake it into
> the ast output and that's way over-engineered.
I guess everyone agrees to raise warning.
("\000" === "\400") === TRUE is wrong, simply.
Regards,
--
Yasuo Ohgaki
[email protected]
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php