Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:105984
MIME-Version: 1.0
References: <e14624ddab96a45b94f4e7504be4c1ac@exussum.co.uk>
 <CAOwTYAtFyfE86qDbXCnwc=Hd9t7RAZsdCOGN=DM3xNZ95dJoqA@mail.gmail.com>
 <e427814a03f4781dd41eb50cf9d074ff@exussum.co.uk> <CAOwTYAsK2c0eDaatWzo-KujydA+AyE_7vG-G9KVA5xjPe087Cw@mail.gmail.com>
In-Reply-To: <CAOwTYAsK2c0eDaatWzo-KujydA+AyE_7vG-G9KVA5xjPe087Cw@mail.gmail.com>
Date: Wed, 19 Jun 2019 10:00:52 +0200
Message-ID: <CAOwTYAtaEMDb3c9Z9TY3W+cxKinzABFBtgh_NfJ0MU573CvK-A@mail.gmail.com>
To: Scott Dutton <scott@exussum.co.uk>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000080127a058ba8a104"
Subject: Re: [PHP-DEV] [RFC] [Vote] Base convert changes
From: krakjoe@gmail.com (Joe Watkins)

--00000000000080127a058ba8a104
Content-Type: text/plain; charset="UTF-8"

It will likely be easier if the Allow negative numbers implementation
(targetting master only) is entirely separate from the Error on ignored
characters implementation,.

Cheers
Joe

On Wed, 19 Jun 2019 at 09:56, Joe Watkins <krakjoe@gmail.com> wrote:

> There should probably be a PR targeting 7.4 with the implementation of
> "Error on ignored characters" as proposed for 7.4, and a PR targeting
> master implementing "Error on ignored characters" with exception change and
> implementing "Allow negative arguments".
>
> None of these PR's should cause tests to fail, and where new untested
> behaviour is introduced the PR should include tests for that.
>
> Cheers
> Joe
>
>
>
> On Wed, 19 Jun 2019 at 09:43, Scott Dutton <scott@exussum.co.uk> wrote:
>
>> Hi Joe,
>>
>> I will take a look at conflicts. The failures are extreme value checks
>> which are a result of allowing the negative numbers. If the negative
>> numbers one passes I will fix all tests and add some more for the
>> negative values. The tests fail because of the unsigned -> signed change
>> (but as you say there were quite a lot of tests).
>>
>> Would it be easier for 2 prs ? one for each vote ?
>>
>> Thanks
>>
>> Scott
>>
>>
>> On 19.06.2019 08:31, Joe Watkins wrote:
>> > The implementation of this does not look ready, there are conflicts
>> > so I can't test it locally, but last time CI ran there were many
>> > failures.
>> >
>> > Cheers
>> > Joe
>> >
>> > On Wed, 19 Jun 2019 at 09:24, Scott Dutton <scott@exussum.co.uk>
>> > wrote:
>> >
>> >> Hi all
>> >>
>> >> I have put my RFC base convert changes to vote this morning
>> >>
>> >> https://wiki.php.net/rfc/base_convert_improvements [1]
>> >>
>> >> Two votes, one to raise a deprecated error in PHP7.4 (raised to
>> >> exception in PHP 8) when base_convert encounters something it doesnt
>> >> know how to convert.
>> >>
>> >> Second vote is to allow negative numbers, eg base_convert('-FF', 16,
>> >> 10) would return -255 (this returns 255 currently)
>> >>
>> >> Voting ends 3rd July
>> >>
>> >> Thanks
>> >>
>> >> Scott
>> >>
>> >> --
>> >> PHP Internals - PHP Runtime Development Mailing List
>> >> To unsubscribe, visit: http://www.php.net/unsub.php [2]
>> >
>> >
>> > Links:
>> > ------
>> > [1] https://wiki.php.net/rfc/base_convert_improvements
>> > [2] http://www.php.net/unsub.php
>>
>

--00000000000080127a058ba8a104--