Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112094
MIME-Version: 1.0
References: <CAFPFaMJMQVVrvBud8dBg=xhGqb5xPyDmUBdUstSeuUfZwZ8vyA@mail.gmail.com>
In-Reply-To: <CAFPFaMJMQVVrvBud8dBg=xhGqb5xPyDmUBdUstSeuUfZwZ8vyA@mail.gmail.com>
Date: Wed, 21 Oct 2020 14:47:42 -0500
Message-ID: <CAESVnVqe20mZAFtTCNph-v07koETo6Rt18j9kG-9JZEyTXwSGw@mail.gmail.com>
To: "G. P. B." <george.banyard@gmail.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000008db81605b2339fca"
Subject: Re: [PHP-DEV][RFC] Add support for explicit octal notation for
 integer literals
From: pollita@php.net (Sara Golemon)

--0000000000008db81605b2339fca
Content-Type: text/plain; charset="UTF-8"

On Wed, Oct 21, 2020 at 9:59 AM G. P. B. <george.banyard@gmail.com> wrote:

> A rather short RFC about adding support for the "0o" prefix for octal
> integers.
> https://wiki.php.net/rfc/explicit_octal_notation
>
> Surprisingly PHP already accepts the prefix within octdec() and
> base_convert().
>
> I have hopefully covered all the cases where this may apply but if you find
> another one please let me know.
>
>
+1

As for deprecating the old syntax in the future, I'd be in favor of that.
I've been bit by this subtle number format a couple of times and writing a
tool to auto-convert to the new format is trivial.  Here's about 3 minutes
worth of effort, for example:
https://gist.github.com/sgolemon/6f9fbb3a95352cbab9297319022d19a5

-Sara

--0000000000008db81605b2339fca--