cvs commit: src/include _ctype.h

Christoph Mallon christoph.mallon at gmx.de
Wed Oct 31 19:02:47 PDT 2007


Andrey Chernov wrote:
> On Wed, Oct 31, 2007 at 10:33:49PM +0000, Alexey Dokuchaev wrote:
>>> For ones who doubts there two tests compiled with -O2. As you may see the 
>>> result is almost identical (andl vs cmpl):
>> Q.E.D.  How about to restore original, more reader-friendly version then?
> 
> 1. Reader-friendly version generates long code when absolutely no 
> optimization used in compiler (for some reason f.e. to avoid optimization 
> bugs).

Code with "absolutely no optimization" is slow for many other reasons.

> 2. It also breaks common style ctype using for is{w}ascii(). If revert 
> this, is{w}ascii() should be rewritted too.

I think, the latter is a good suggestion.

	Christoph


More information about the cvs-src mailing list