Re: svn: /php/php-src/branches/PHP_5_3/ext/standard/ basic_functions.c http.c php_http.h url.h
| From: | Pierre Joye | Date: | Mon, 10 Jan 2011 08:20:23 +0000 |
| Subject: | Re: svn: /php/php-src/branches/PHP_5_3/ext/standard/ basic_functions.c http.c php_http.h url.h | ||
| References: | 1 2 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-64324@lists.php.net to get a copy of this message | ||
hi,
Yes, it breaks the ABI, please revert this patch.
Maybe another implementation by adding a new function could help, but
I'm not sure we need that now in 5.3 either.
Cheers,
On Mon, Jan 10, 2011 at 5:35 AM, Adam Harvey <aharvey@php.net> wrote:
> On 8 January 2011 10:34, Rui Hirokawa <hirokawa@php.net> wrote:
>> hirokawa Sat, 08 Jan 2011 02:34:33 +0000
>>
>> Revision:
>> http://svn.php.net/viewvc?view=revision&revision=307248
>>
>> Log:
>> MFH: added an option to http_build_query for RFC-3986
>> based url-encoding.
>
> Somebody should double check me on this, because binary
> compatibility's not one of my stronger points, but isn't the change to
> the php_url_encode_hash_ex() definition going to break BC in 5.3?
> ext/standard/php_http.h is one of the headers we install.
>
> Adam
>
> --
> PHP CVS Mailing List (http://www.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
--
Pierre
@pierrejoye | http://blog.thepimp.net | http://www.libgd.org