Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2(PHP_5_3) / zend_operators.h php-src configure.in

<tony2001> it's a GNU extension, hence you need a define to make it available
<tony2001> this define is different everywhere
<tony2001> glibc in SuSE uses __USE_GNU, glibc in Ubuntu uses __STANDARD_LIB
<nlopes> uhm, my man pages says it's always defined (despite being a GNU extension)
<tony2001> Also native memchr() didn't work for me on 64bit Linux - it was returning some garbage pointer instead of valid string
<nlopes> but that's a bug in glibc or gcc..
<tony2001> nlopes: you know how bugs in glibc are dealt with..
<nlopes> upgrade it! :)
<tony2001> so currently I see no reasons to use memrchr() as long as we have zend_memrchr()

On 13.02.2008 19:30, Antony Dovgal wrote:
> We've already had this fallback and it's been reverted because it didn't quite work.
> 
> http://cvs.php.net/viewvc.cgi/ZendEngine2/zend_operators.h?r1=1.111&r2=1.112
> 
> On 13.02.2008 19:22, Dmitry Stogov wrote:
>> dmitry		Wed Feb 13 16:22:01 2008 UTC
>> 
>>   Modified files:              (Branch: PHP_5_3)
>>     /php-src	configure.in 
>>     /ZendEngine2	zend_operators.h 
>>   Log:
>>   Use memrchr() instead of zend_memrchr() if available. (Nuno)
> 


-- 
Wbr, 
Antony Dovgal