From: Jeff King Date: Mon, 13 Jun 2016 05:39:20 GMT Subject: [PATCH 2/3] interpret-trailers: don't duplicate option strings Message-ID: <20160613053920.GB23880@sigill.intra.peff.net> In-Reply-To: <20160613053203.GB3950@sigill.intra.peff.net> There's no need to do so; the argv strings will last until the end of the program. Signed-off-by: Jeff King --- builtin/interpret-trailers.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/interpret-trailers.c b/builtin/interpret-trailers.c index b99ae4b..175f147 100644 --- a/builtin/interpret-trailers.c +++ b/builtin/interpret-trailers.c @@ -20,7 +20,7 @@ int cmd_interpret_trailers(int argc, const char **argv, const char *prefix) { int in_place = 0; int trim_empty = 0; - struct string_list trailers = STRING_LIST_INIT_DUP; + struct string_list trailers = STRING_LIST_INIT_NODUP; struct option options[] = { OPT_BOOL(0, "in-place", &in_place, N_("edit files in place")), -- 2.9.0.rc2.149.gd580ccd