Re: [PHP-DEV] [RFC] [Vote] Base convert changes

2019-07-02 Thread Nikita Popov
On Mon, Jul 1, 2019 at 4:09 PM Scott Dutton wrote: > Hi all > > I have made a PR for 7.4 for just the ignoring invalid input. I will > open a similar one for PHP 8. > > https://github.com/php/php-src/pull/4328 > > All tests have been fixed for linux, there is one failure for windows > which I am

RE: [PHP-DEV] [RFC] [Vote] Base convert changes

2019-07-01 Thread Scott Dutton
Hi all I have made a PR for 7.4 for just the ignoring invalid input. I will open a similar one for PHP 8. https://github.com/php/php-src/pull/4328 All tests have been fixed for linux, there is one failure for windows which I am looking in to. Voting ends in a few days time so last chance

RE: [PHP-DEV] [RFC] [Vote] Base convert changes

2019-06-20 Thread Scott Dutton
#x27;t find it in the RFC. e.g. base_convert('--ff', 16, 10) Thanks, CHU Zhaowei -Original Message- From: Scott Dutton Sent: Wednesday, June 19, 2019 3:24 PM To: internals@lists.php.net Subject: [PHP-DEV] [RFC] [Vote] Base convert changes Hi all I have put my RFC base convert

RE: [PHP-DEV] [RFC] [Vote] Base convert changes

2019-06-19 Thread CHU Zhaowei
als@lists.php.net > Subject: [PHP-DEV] [RFC] [Vote] Base convert changes > > Hi all > > I have put my RFC base convert changes to vote this morning > > https://wiki.php.net/rfc/base_convert_improvements > > Two votes, one to raise a deprecated error in PHP7.4 (raised to

Re: [PHP-DEV] [RFC] [Vote] Base convert changes

2019-06-19 Thread Scott Dutton
Ah thats brilliant, I will take a look at updating the tests when I split the PR tomorrow Thanks Nikita Scott On 19.06.2019 09:13, Nikita Popov wrote: On Wed, Jun 19, 2019 at 10:06 AM Scott Dutton wrote: Hi Joe, I will have a look at splitting the PR, I am not at a computer where i c

Re: [PHP-DEV] [RFC] [Vote] Base convert changes

2019-06-19 Thread Nikita Popov
On Wed, Jun 19, 2019 at 10:06 AM Scott Dutton wrote: > Hi Joe, > > I will have a look at splitting the PR, I am not at a computer where i > can code today though so will be tomorrow at the earliest. > > The Negative numbers will be a fair amount of work to make the tests > pass, would this still

Re: [PHP-DEV] [RFC] [Vote] Base convert changes

2019-06-19 Thread Scott Dutton
Hi Joe, I will have a look at splitting the PR, I am not at a computer where i can code today though so will be tomorrow at the earliest. The Negative numbers will be a fair amount of work to make the tests pass, would this still need to be done if the RFC doesnt pass ? I am happy to do this

Re: [PHP-DEV] [RFC] [Vote] Base convert changes

2019-06-19 Thread Joe Watkins
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 wrote: > There should probably be a PR targeting 7.4 with the implementa

Re: [PHP-DEV] [RFC] [Vote] Base convert changes

2019-06-19 Thread Joe Watkins
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

Re: [PHP-DEV] [RFC] [Vote] Base convert changes

2019-06-19 Thread Scott Dutton
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 a

Re: [PHP-DEV] [RFC] [Vote] Base convert changes

2019-06-19 Thread Joe Watkins
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 wrote: > Hi all > > I have put my RFC base convert changes to vote this morning > > https://wiki.p

[PHP-DEV] [RFC] [Vote] Base convert changes

2019-06-19 Thread Scott Dutton
Hi all I have put my RFC base convert changes to vote this morning https://wiki.php.net/rfc/base_convert_improvements 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 a