[PATCH 3/3] get_urlmatch: avoid useless strbuf write
- From
Jeff King <peff@peff.net>
- Date
- Aug 20, 2015, 14:49 UTC
- Message-ID
- <20150820144945.GC11913@sigill.intra.peff.net>
- In-Reply-To
- <20150820144504.GA22935@sigill.intra.peff.net>
We create a strbuf only to insert a single string, pass the resulting buffer to a function (which does not modify the string), and then free it. We can just pass the original string instead.
Signed-off-by: Jeff King <peff@peff.net> --- I keep staring at this thinking I missed something, but I think this is an equivalent transformation. I wonder if the original meant to modify the key in some way, but I don't see how or why.
builtin/config.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/builtin/config.c b/builtin/config.c index 04befce..71acc44 100644 --- a/builtin/config.c +++ b/builtin/config.c @@ -425,14 +425,11 @@ static int get_urlmatch(const char *var, const char *url) for_each_string_list_item(item, &values) { struct urlmatch_current_candidate_value *matched = item->util; - struct strbuf key = STRBUF_INIT; struct strbuf buf = STRBUF_INIT; - strbuf_addstr(&key, item->string); - format_config(&buf, key.buf, + format_config(&buf, item->string, matched->value_is_null ? NULL : matched->value.buf); fwrite(buf.buf, 1, buf.len, stdout); - strbuf_release(&key); strbuf_release(&buf); strbuf_release(&matched->value);
-- 2.5.0.680.g69e7703