Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:105985 Return-Path: Delivered-To: mailing list internals@lists.php.net Received: (qmail 75988 invoked from network); 19 Jun 2019 10:51:13 -0000 Received: from unknown (HELO thephantoms.co.uk) (176.126.243.54) by pb1.pair.com with SMTP; 19 Jun 2019 10:51:13 -0000 Comment: DKIM? See http://www.dkim.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=exussum.co.uk; s=default; x=1561536349; h=Comment: DomainKey-Signature:Received:To:Subject:MIME-Version: Content-Type:Content-Transfer-Encoding:Date:From:Cc:In-Reply-To: References:Message-ID:User-Agent; bh=1SGVzVgZaWKaIeawi4ynSuW8bSo 5wnG46nS8wcay1po=; b=ZLMfez7I/wTW99cZhNpuk3S80UlqZe+6mg8qSoGZK7J 0xlqpsH7aITn4CzgmOcaqjN8O0HJSqPiQMDkqxUAgL5eANlSfw61H+hqeYwE3OZY NsFpWA1S/EGrY8pN6053YyNQ1Vm4QkxVGrSqzuJZKQP8EcyQ2O07T8UU2ll7C0mg = Comment: DomainKeys? See http://antispam.yahoo.com/domainkeys DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=default; d=exussum.co.uk; h=Received:To:Subject:MIME-Version:Content-Type:Content-Transfer-Encoding:Date:From:Cc:In-Reply-To:References:Message-ID:X-Sender:User-Agent; b=dsS/Hw51nNuETWbG6O98pxJnF3RgwUZEPi/kveaLQoxV4rSYhc2mq4hADXaYF5 nco+mKByfzNS5hFHJx6EBRIDeA1HEw1FwsRX0j+duWkXzim0ZzY1y9sJxPB1sjV+ lFGA+zuP24gWLzok8LWvMBP7cgbL5jze4tl0nGuT3im9I=; Received: (qmail 23791 invoked by uid 48); 19 Jun 2019 08:05:48 -0000 To: Joe Watkins MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Date: Wed, 19 Jun 2019 09:05:48 +0100 Cc: PHP internals In-Reply-To: References: Message-ID: X-Sender: scott@exussum.co.uk User-Agent: RoundCube Webmail/0.1 Subject: Re: [PHP-DEV] [RFC] [Vote] Base convert changes From: scott@exussum.co.uk (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 work if it looks like it will pass, the reasons it fails are outlined in the RFC as BC breaks. The tests seem to test the values which make it fail more than I have seen other code use these values. 887-939 are ignoring invalid input changes, everything else is negative numbers Hope that helps Scott On 19.06.2019 08:56, Joe Watkins 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 > 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 >> > wrote: >> > >> >> Hi all >> >> >> >> I have put my RFC base convert changes to vote this morning >> >> >> >> https://wiki.php.net/rfc/base_convert_improvements [1] [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] [2] >> > >> > >> > Links: >> > ------ >> > [1] https://wiki.php.net/rfc/base_convert_improvements [1] >> > [2] http://www.php.net/unsub.php [2] > > > Links: > ------ > [1] https://wiki.php.net/rfc/base_convert_improvements > [2] http://www.php.net/unsub.php