Re: svn: /php/php-src/trunk/ NEWS Zend/zend_compile.c Zend/zend_globals.h Zend/zend_multibyte.c Zend/zend_multibyte.h ext/exif/exif.c
ext/mbstring/mbstring.c ext/standard/html.c main/rfc1867.c main/rfc1867.h tests/basic/028.phpt tests/basic

On 12/09/2010 08:51 PM, Moriyoshi Koizumi wrote:

I was thinking that this is rather a *big* change, so we'd better roll our patches into one and commit it altogether.

Sorry, that time I didn't understand that you started to improve the patch.

I am fine if that works for anyone else.

The patch had few small issues, but they are already fixed. Thanks. Dmitry.

Regards, Moriyoshi On Thu, Dec 9, 2010 at 6:11 PM, Dmitry Stogov<dmitry@zend.com> wrote:

Hi Moriyoshi, Ops, sorry. I didn't understand your disagreement, and I didn't see any other objections. So I committed the patch as it was planned. Could you prepare a patch on top of this one? I'm also going to look into your recommendation of moving mbstring.script_encoding into zend.script_encoding instead of zend.multibyte. Thanks. Dmitry. On 12/09/2010 09:02 AM, Moriyoshi Koizumi wrote:

Please revert.

On Thu, Dec 9, 2010 at 3:00 PM, Moriyoshi Koizumi<mozo@mozo.jp>    wrote:

Hey, I don't think there was agreement on this yet. I've actually been preparing a patch that minimizes the extra hook points in SAPI. Moriyoshi

On Wed, Dec 8, 2010 at 8:27 PM, Dmitry Stogov<dmitry@php.net>    wrote:

dmitry                                   Wed, 08 Dec 2010 11:27:34 +0000

Revision: http://svn.php.net/viewvc?view=revision&revision=306077 Log: Removed compile time dependency from ext/mbstring Changed paths:

    U   php/php-src/trunk/NEWS
    U   php/php-src/trunk/Zend/zend_compile.c
    U   php/php-src/trunk/Zend/zend_globals.h
    U   php/php-src/trunk/Zend/zend_multibyte.c
    U   php/php-src/trunk/Zend/zend_multibyte.h
    U   php/php-src/trunk/ext/exif/exif.c
    U   php/php-src/trunk/ext/mbstring/mbstring.c
    U   php/php-src/trunk/ext/standard/html.c
    U   php/php-src/trunk/main/rfc1867.c
    U   php/php-src/trunk/main/rfc1867.h
    A   php/php-src/trunk/tests/basic/028.phpt
    A   php/php-src/trunk/tests/basic/029.phpt

-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php