On Sun, Feb 24, 2008 at 08:42:26AM +0000, Junio C Hamano wrote: > Pierre Habouzit <madcoder@xxxxxxxxxx> writes: > > > I also made the parsing more extensible wrt new flags if needed. > > > > We should definitely write tests too. > > ... > > + case '*': > > + o->flags &= PARSE_OPT_HIDDEN; > > + break; > > I have a slight suspicion that this is not what you meant. > You probably meant either |= or perhaps ^=. oooh boy of course it's a |=... -- ·O· Pierre Habouzit ··O madcoder@xxxxxxxxxx OOO http://www.madism.org
Attachment:
pgpGCUQ40chHP.pgp
Description: PGP signature