From: Timo Hirvonen <tihirvon@gmail.com>
Date: Sun, 18 Jun 2006 21:24:45 GMT
Subject: Re: [PATCH 1/7] Remove ranges from switch statements.
Message-ID: <20060619002445.3af68c2b.tihirvon@gmail.com>
In-Reply-To: <7vveqyyxyj.fsf@assigned-by-dhcp.cox.net>

Junio C Hamano <junkio@cox.net> wrote:

> Sorry for bringing up an old topic again, but wouldn't people
> agree that this is easier to read if it were written this way ;-)?
>  
> 	if (	   (('A' <= ch) && (ch <= 'Z'))
>         	|| (('a' <= ch) && (ch <= 'z'))
> 		|| (('0' <= ch) && (ch <= '9'))
> 	...

Yes, but isalnum(ch) even better ;)

-- 
http://onion.dynserv.net/~timo/