On Sat, Sep 29, 2007 at 12:52:36PM +0000, David Kastrup wrote: > Pierre Habouzit <madcoder@xxxxxxxxxx> writes: > > > Signed-off-by: Pierre Habouzit <madcoder@xxxxxxxxxx> > > > - if (strcmp(argv[i], "-z")) > > - revs->diff = 1; > > + revs->diff = strcmp(argv[i], "-z") > > + && strcmp(argv[i], "--color") > > + && strcmp(argv[i], "--no-color"); > > i += opts - 1; > > continue; > > This can clear a previously set value of revs->diff. Good catch, that should be |= of course. -- ·O· Pierre Habouzit ··O madcoder@xxxxxxxxxx OOO http://www.madism.org
Attachment:
pgppWs241RDyO.pgp
Description: PGP signature