[PHP-CVS] cvs: php-src(PHP_5_3) /ext/mbstring/libmbfl/filters mbfilter_cp936.c /ext/mbstring/tests bug46843.phpt

[Moriyoshi Koizumi]

moriyoshi               Mon Feb  2 04:53:10 2009 UTC

  Added files:                 (Branch: PHP_5_3)
    /php-src/ext/mbstring/tests bug46843.phpt 

  Modified files:              
    /php-src/ext/mbstring/libmbfl/filters       mbfilter_cp936.c 
  Log:
  - MFH: Fix bug #46843 (CP936 euro symbol is not converted properly)
  
  
http://cvs.php.net/viewvc.cgi/php-src/ext/mbstring/libmbfl/filters/mbfilter_cp936.c?r1=1.1&r2=1.1.10.1&diff_format=u
Index: php-src/ext/mbstring/libmbfl/filters/mbfilter_cp936.c
diff -u php-src/ext/mbstring/libmbfl/filters/mbfilter_cp936.c:1.1 
php-src/ext/mbstring/libmbfl/filters/mbfilter_cp936.c:1.1.10.1
--- php-src/ext/mbstring/libmbfl/filters/mbfilter_cp936.c:1.1   Sat Aug 23 
06:18:36 2003
+++ php-src/ext/mbstring/libmbfl/filters/mbfilter_cp936.c       Mon Feb  2 
04:53:10 2009
@@ -192,7 +192,7 @@
                }
        }
        if (s >= 0) {
-               if (s < 0x80) { /* latin */
+               if (s <= 0x80) {        /* latin */
                        CK((*filter->output_function)(s, filter->data));
                } else {
                        CK((*filter->output_function)((s >> 8) & 0xff, 
filter->data));

http://cvs.php.net/viewvc.cgi/php-src/ext/mbstring/tests/bug46843.phpt?view=markup&rev=1.1
Index: php-src/ext/mbstring/tests/bug46843.phpt
+++ php-src/ext/mbstring/tests/bug46843.phpt
--TEST--
Bug #46843 (CP936 euro symbol is not converted properly)
--SKIPIF--
<?php extension_loaded('mbstring') or die('skip mbstring not available'); ?>
--FILE--
<?php
var_dump(bin2hex(mb_convert_encoding("\x80", 'UCS-2BE', 'CP936')));
var_dump(bin2hex(mb_convert_encoding("\x20\xac", 'CP936', 'UCS-2BE')));
?>
--EXPECT--
string(4) "20ac"
string(2) "80"



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