Date: Tue, 30 Oct 2007 06:44:46 +0000 From: Alexey Dokuchaev <danfe@FreeBSD.org> To: Christoph Mallon <christoph.mallon@gmx.de> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, "Andrey A. Chernov" <ache@FreeBSD.org>, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/include _ctype.h Message-ID: <20071030064446.GC68358@FreeBSD.org> In-Reply-To: <47264710.2000500@gmx.de> References: <200710272232.l9RMWSbK072082@repoman.freebsd.org> <47264710.2000500@gmx.de>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Oct 29, 2007 at 09:48:16PM +0100, Christoph Mallon wrote: > Andrey A. Chernov wrote: > >ache 2007-10-27 22:32:28 UTC > > > > FreeBSD src repository > > > > Modified files: > > include _ctype.h > > Log: > > Micro-optimization of prev. commit, change > > (_c < 0 || _c >= 128) to (_c & ~0x7F) > > > > Revision Changes Path > > 1.33 +1 -1 src/include/_ctype.h > > Further IMO it is hard to decipher what _c & ~0x7F is supposed to do. Seconded. Messing with bits should be avoided when their use masks out the logic behind. Nowadays it's almost always better to trust the compiler to do its optimizing job. ./danfe
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20071030064446.GC68358>