Re: [RFC] [Vote] Base convert changes
| From: | Scott Dutton | Date: | Wed, 19 Jun 2019 07:43:33 +0000 |
| Subject: | Re: [RFC] [Vote] Base convert changes | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-105982@lists.php.net to get a copy of this message | ||
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