Re: --enable-zend-multibyte

From:	Dmitry Stogov	Date:	Fri, 19 Nov 2010 14:08:23 +0000
Subject:	Re: --enable-zend-multibyte
References:	1 2	Groups:	php.internals
Request:	Send a blank email to internals+get-50385@lists.php.net to get a copy of this message

CG(multibyte) is defined only inside #ifdef ZEND_MULTIBYTE. Thanks. Dmitry. Patrick ALLAERT wrote:

2010/11/18 Dmitry Stogov <dmitry@zend.com>:
Index: ext/mbstring/mbstring.c ===================================================================
--- ext/mbstring/mbstring.c     (revision 305494)
+++ ext/mbstring/mbstring.c     (working copy)
@@ -1132,6 +1132,9 @@ {
       int *list, size;
+       if (!CG(multibyte)) {
+               return FAILURE;
+       }
       if (php_mb_parse_encoding_list(new_value, new_value_length, &list,
&size, 1 TSRMLS_CC)) {
               if (MBSTRG(script_encoding_list) != NULL) {
                       free(MBSTRG(script_encoding_list));
Should this hunk not be inside an #ifdef ZEND_MULTIBYTE ?