Re: [PATCH 1/7] Remove ranges from switch statements.
- From
- Timo Hirvonen <tihirvon@gmail.com>
- Date
- Jun 18, 2006, 21:24 UTC
- Message-ID
- <20060619002445.3af68c2b.tihirvon@gmail.com>
- In-Reply-To
- <7vveqyyxyj.fsf@assigned-by-dhcp.cox.net>
Junio C Hamano <junkio@cox.net> wrote:
Show 7 quoted lines
> 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/