On Tue, Jun 13, 2017 at 01:42:02PM +0200, Johannes Schindelin wrote: > > As you probably guessed, I had tried that first and then figured that if > > I needed to keep the config_key_is_valid() test anyway, I could just as > > well keep the strbuf around for later use. > > > > Will change the code, > > Alas, I won't change the code after all. > > It is really tempting to avoid the extra strbuf, but then the error > message would change from > > error: missing value for 'alias.br' > > to > > error: missing value for 'br' > > which is of course no good at all. > > And since I already have to keep that strbuf, I'll simply keep the > config_key_is_valid() guard, too (because why not). Oof, yeah, that is definitely worse. I'm fine with keeping both parts. -Peff