Re: svn: /php/php-src/branches/PHP_5_3/ext/standard/ basic_functions.c http.c php_http.h url.h

From: 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

« previous php.cvs (#64324) next »