From: Andy Isaacson <adi@hexapodia.org>
Date: Fri, 22 Apr 2005 23:30:12 GMT
Subject: Re: [patch] fixup GECOS handling
Message-ID: <20050422233012.GA27638@hexapodia.org>
In-Reply-To: <1114186599.31076.409.camel@axer.marchex.com>

On Fri, Apr 22, 2005 at 09:16:39AM -0700, Kyle Hayes wrote:
> if(comma)
> 	if(semi)
> 		/* lastname, firstname; room #; phone # format */
> 		*semi  = 0;
> 	else
> 		*comma = 0;
> else if(semi)
> 	*semi = 0;

That's a really complicated way of writing

if(semi) *semi = 0;
else if(comma) *comma = 0;

(The two code fragments are precisely identical.  Mmmm, strength
reduction.)

-andy