Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:104970
MIME-Version: 1.0
References: <php.internals-104590@news.php.net> <cee93791-8516-4653-9d8e-af70bd7e1a0c@BL2NAM02FT006.eop-nam02.prod.protection.outlook.com>
 <DM5PR06MB2857BD900DFC1510DD3EA314DE5F0@DM5PR06MB2857.namprd06.prod.outlook.com>
 <CAG9XoMSktgyZ_634JyOEU_ov-kEjhhEWaxEEo31th5x+0f8M1A@mail.gmail.com>
 <b4752b1a-c89d-0f4b-dd20-48cfe3747677@gmail.com> <CAG9XoMRnK+ceLHa5kHQkZRF9w3pEhmpoVaW2dEoaHRHdOdDHCQ@mail.gmail.com>
In-Reply-To: <CAG9XoMRnK+ceLHa5kHQkZRF9w3pEhmpoVaW2dEoaHRHdOdDHCQ@mail.gmail.com>
Date: Wed, 27 Mar 2019 16:19:31 +0000
Message-ID: <CALKiJKqitXeujSotXBwxEdBCPZ4TQhbR4LX=DSwna=tZiHXzTw@mail.gmail.com>
To: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000251c44058515ce08"
Subject: Re: [PHP-DEV] Re: [RFC] Permit trailing whitespace in numeric strings
From: rowan.collins@gmail.com (Rowan Collins)

--000000000000251c44058515ce08
Content-Type: text/plain; charset="UTF-8"

On Tue, 26 Mar 2019 at 23:09, Benjamin Morel <benjamin.morel@gmail.com>
wrote:

> Also, may I suggest that these 2 RFCs be merged together?
>
> https://wiki.php.net/rfc/string_to_number_comparison
> https://wiki.php.net/rfc/trailing_whitespace_numerics
>


Although they are complementary, the two RFCs are tackling different
subjects, so should remain separate IMHO.

- Andrea's RFC is discussing the behaviour of parsing number strings in a
wide variety of contexts, including implicit and explicit casts.
- Nikita's RFC is discussing the specific case of comparisons, and applies
existing number-parsing rules to different contexts

I am certainly in favour of them both targeting PHP 8, possibly along with
additional "type juggling" changes, which could all be documented together,
but I think it would just be confusing to try to cover all the examples,
pros and cons, and voting options in one RFC.

Regards,
-- 
Rowan Collins
[IMSoP]

--000000000000251c44058515ce08--