Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:105983
MIME-Version: 1.0
References: <e14624ddab96a45b94f4e7504be4c1ac@exussum.co.uk>
 <CAOwTYAtFyfE86qDbXCnwc=Hd9t7RAZsdCOGN=DM3xNZ95dJoqA@mail.gmail.com> <e427814a03f4781dd41eb50cf9d074ff@exussum.co.uk>
In-Reply-To: <e427814a03f4781dd41eb50cf9d074ff@exussum.co.uk>
Date: Wed, 19 Jun 2019 09:56:46 +0200
Message-ID: <CAOwTYAsK2c0eDaatWzo-KujydA+AyE_7vG-G9KVA5xjPe087Cw@mail.gmail.com>
To: Scott Dutton <scott@exussum.co.uk>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000d9d25d058ba892b0"
Subject: Re: [PHP-DEV] [RFC] [Vote] Base convert changes
From: krakjoe@gmail.com (Joe Watkins)

--000000000000d9d25d058ba892b0
Content-Type: text/plain; charset="UTF-8"

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
>

--000000000000d9d25d058ba892b0--