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


Thread (132 messages)

« previous php.internals (#71727) next »