From: Kyle Hayes Date: Fri, 22 Apr 2005 17:58:22 GMT Subject: Re: [patch] fixup GECOS handling Message-ID: <1114192702.31076.428.camel@axer.marchex.com> In-Reply-To: <20050422171818.GE7173@pasky.ji.cz> On Fri, 2005-04-22 at 19:18 +0200, Petr Baudis wrote: > Dear diary, on Fri, Apr 22, 2005 at 06:58:25PM CEST, I got a letter > where Martin Schlemmer told me that... > > Meaning, if they use a ',' in one of the fields (and it is a linux > > system with the chfn most probably from the shadow package), then they > > are looking for trouble. The only reason I added the ';' was because > > somebody said whatever OS used it instead of a ','. > > What about just swapping the two tests so that ; is cut off and , only > when no ; is around? Even nicer. I like it. Very clean! Best, Kyle -- Kyle Hayes Marchex Inc.