On Fri, Aug 02, 2019 at 09:53:55AM -0700, Junio C Hamano wrote: > Jonathan Nieder <jrnieder@xxxxxxxxx> writes: > > > We'll also want to update the docs. And as Todd suggests, we should > > cover how to disable mailmap in tests. > > > > Signed-off-by: Jonathan Nieder <jrnieder@xxxxxxxxx> > > --- > > I avoided the "don't bother initializing use_mailmap_config to > unknown" simplification, but I guess the change is clear enough. Yeah, I didn't even look to see if there were further simplifications possible, but seeing Jonathan's patch, it looks like an obvious improvement. And a quick grep shows that there are no other sites that should be affected by losing the "-1" semantics, so I think it's a good idea. And of course the doc and test improvements are very welcome. :) -Peff
