Re: [VOTE] 64 bit platform improvements for string length and integer

From: Date: Wed, 29 Jan 2014 15:00:21 +0000
Subject: Re: [VOTE] 64 bit platform improvements for string length and integer
References: 1  Groups: php.internals 
Request: Send a blank email to internals+get-71727@lists.php.net to get a copy of this message
Nikita, On Tue, January 28, 2014 21:51, Nikita Popov wrote: > On Tue, Jan 28, 2014 at 9:33 PM, Pierre Joye <pierre.php@gmail.com> > wrote: > > >>> Just to clarify my vote: I voted "No" on the RFC, but only because I >>> >> think >>> the major extension source API break is better suited to PHP 6 than >>> PHP >>> 5.6. I fully support merging this into master, as the first change for >>> >> PHP >> >>> 6. >>> >> >> Sad. As the work for pecl extensions is really not as large as you >> think, fairly straight forward. The vote for php6 is not running now. >> There is no option for PHP 6 as what can be done in php 6 internally >> is much larger and would allow more breakage. On the other side, we >> provide options to minimize the work for the extensions developers in >> order of magnitude smaller than the changes we introduced in the past >> for the OO APIs for example. Anyway, sad that you voted no for >> something as important than true 64bit support. >> > > Maybe I'm indeed overestimating the impact. Could you please link to the > source-code for a PECL (or other third-party) extension that was ported to > be compatible with the new API, so people can see the actual impact this > makes? (Hopefully some ext that requires compat across multiple versions > like 5.3+ and has non-trivial amount of code and functions, like a few > dozen.) I couldn't find an example in RFC, but maybe I just missed it. > Only > saw small snippets in there. > please look here https://github.com/weltling/mongo-php-driver/compare/mongodb:master...master two hours of work and it's about 70% of the porting, compiling with 5.5 and str_size_and_int64. This is the hard caliber of course. Thanks Anatol

« previous php.internals (#71727) next »