From: Junio C Hamano Date: Fri, 21 Aug 2015 17:43:58 GMT Subject: Re: [PATCH 2/3] format_config: simplify buffer handling Message-ID: In-Reply-To: Junio C Hamano writes: > I wonder if we can do this instead > > if (!omit_values) { > - if (show_keys) > + if (show_keys && value_) > strbuf_addch(buf, key_delim); > > though. That would eliminate the need for rolling back. No we cannot. "config --bool --get-regexp" could get a NULL value_ and has to turn it to " true". Sorry for the noise.