Date: Wed, 31 Oct 2007 12:43:13 -1000 From: Juli Mallett <juli@clockworksquid.com> To: Andrey Chernov <ache@nagual.pp.ru>, Alexey Dokuchaev <danfe@FreeBSD.ORG>, Juli Mallett <juli@clockworksquid.com>, src-committers@FreeBSD.ORG, cvs-src@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/include _ctype.h Message-ID: <20071031224313.GA18285@toxic.magnesium.net> In-Reply-To: <20071031223727.GB90994@nagual.pp.ru> References: <200710272232.l9RMWSbK072082@repoman.freebsd.org> <20071030200331.GA29309@toxic.magnesium.net> <20071031215526.GC89932@nagual.pp.ru> <20071031223349.GA552@FreeBSD.org> <20071031223727.GB90994@nagual.pp.ru>
next in thread | previous in thread | raw e-mail | index | archive | help
* Andrey Chernov <ache@nagual.pp.ru> [ 2007-10-31 ]
[ Re: cvs commit: src/include _ctype.h ]
> 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).
So if someone is trying to avoid compiler optimization bugs we should subject
them to human optimization bugs instead? :)
> 2. It also breaks common style ctype using for is{w}ascii(). If revert
> this, is{w}ascii() should be rewritted too.
That seems reasonable.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20071031224313.GA18285>