Re: non-functional backout for review

From: David Xu <davidxu_at_FreeBSD.org>
Date: Thu, 01 Nov 2007 11:15:21 +0800
Andrey Chernov wrote:
> Some people complaints that new check in __isctype() is not human readable 
> and requests for backout. Compiler gives almost identical code for old and 
> new excepting non-optimized case where non-human readable one wins. I am a 
> bit tired to change it forth and back, so ask for final consensus here.
> 
> Old one
> 
> return (_c < 0 || _c >= 128) ? 0 :
> 
> New one (requests for backout)
> 
> return (_c & ~0x7F) ? 0 :
> 

I will use a version which compiler can generate optimized code.
the name __isctype already tells what the function is doing. :-)

Regards,
David Xu
Received on Thu Nov 01 2007 - 02:14:30 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:20 UTC