Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/mbstring mbstring.c mbstring.h

2008-07-25 Thread Moriyoshi Koizumi
I overlooked one more error; I wrote --SKIP-- everywhere --SKIPIF-- is expected :/ I committed the fix and now it should be ok ;) Moriyoshi On 2008/07/25, at 19:47, Antony Dovgal wrote: On 25.07.2008 14:36, Moriyoshi Koizumi wrote: ini_set() returns the previously set value, so that's fine.

Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/mbstring mbstring.c mbstring.h

2008-07-25 Thread Antony Dovgal
On 25.07.2008 14:36, Moriyoshi Koizumi wrote: ini_set() returns the previously set value, so that's fine. But I eventually noticed that the logic is inversed - "or" should be "and" actually. Thanks for the heads-up. The tests still fail, though. Even with zend multibyte enabled. -- ext/mbst

Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/mbstring mbstring.c mbstring.h

2008-07-25 Thread Moriyoshi Koizumi
On 2008/07/25, at 19:11, Antony Dovgal wrote: On 25.07.2008 13:51, Moriyoshi Koizumi wrote: It's probable that segfaults occurred in the test 10, 11 and 12. would you mind posting it if that's the case? No, there are no segfaults and valgrind keeps silence, too. It seems I found it: --SKI

Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/mbstring mbstring.c mbstring.h

2008-07-25 Thread Antony Dovgal
On 25.07.2008 13:51, Moriyoshi Koizumi wrote: It's probable that segfaults occurred in the test 10, 11 and 12. would you mind posting it if that's the case? No, there are no segfaults and valgrind keeps silence, too. It seems I found it: --SKIP-- Note that "***" instead of SJIS. -- Wbr, A

Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/mbstring mbstring.c mbstring.h

2008-07-25 Thread Moriyoshi Koizumi
It's probable that segfaults occurred in the test 10, 11 and 12. would you mind posting it if that's the case? Moriyoshi On 2008/07/25, at 18:51, Antony Dovgal wrote: On 25.07.2008 13:29, Moriyoshi Koizumi wrote: Can you send me the test results? I tested on PPC and i386, and they went we

Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/mbstring mbstring.c mbstring.h

2008-07-25 Thread Antony Dovgal
On 25.07.2008 13:29, Moriyoshi Koizumi wrote: Can you send me the test results? I tested on PPC and i386, and they went well without problems. I'm testing on Linux 64bit. Here are the diffs. -- ext/mbstring/tests/zend_multibyte-02.diff -- 001+ string(18) "e38386e382b9e38388" 001- string(12)

Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/mbstring mbstring.c mbstring.h

2008-07-25 Thread Moriyoshi Koizumi
Can you send me the test results? I tested on PPC and i386, and they went well without problems. Moriyoshi On 2008/07/25, at 16:09, Antony Dovgal wrote: On 25.07.2008 02:21, Moriyoshi Koizumi wrote: moriyoshi Thu Jul 24 22:21:42 2008 UTC Added files: (Branch:

Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/mbstring mbstring.c mbstring.h

2008-07-25 Thread Antony Dovgal
On 25.07.2008 02:21, Moriyoshi Koizumi wrote: moriyoshi Thu Jul 24 22:21:42 2008 UTC Added files: (Branch: PHP_5_3) /php-src/ext/mbstring/tests zend_multibyte-01.phpt zend_multibyte-02.phpt zend