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<m...@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.


On Wed, Dec 8, 2010 at 8:27 PM, Dmitry Stogov<dmi...@php.net>  wrote:
dmitry                                   Wed, 08 Dec 2010 11:27:34 +0000

Revision: http://svn.php.net/viewvc?view=revision&revision=306077

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

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

Reply via email to