diff options
| author |  Junio C Hamano <gitster@pobox.com> | 2016-07-06 13:38:07 -0700 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2016-07-06 13:38:08 -0700 |
| commit | b8b6365a8a214f2427f2cf9334fd43ace0be619c (patch) | |
| tree | 983ce79cdb05e7d8612b3240699b9cbf20370a30 /parse-options-cb.c | |
| parent | Merge branch 'jk/send-pack-stdio' (diff) | |
| parent | use string_list initializer consistently (diff) | |
| download | tgif-b8b6365a8a214f2427f2cf9334fd43ace0be619c.tar.xz | |
Merge branch 'jk/string-list-static-init'
Instead of taking advantage of a struct string_list that is
allocated with all NULs happens to be STRING_LIST_INIT_NODUP kind,
initialize them explicitly as such, to document their behaviour
better.
* jk/string-list-static-init:
use string_list initializer consistently
blame,shortlog: don't make local option variables static
interpret-trailers: don't duplicate option strings
parse_opt_string_list: stop allocating new strings
Diffstat (limited to 'parse-options-cb.c')
| -rw-r--r-- | parse-options-cb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/parse-options-cb.c b/parse-options-cb.c index 239898d946..ba5acf3111 100644 --- a/parse-options-cb.c +++ b/parse-options-cb.c @@ -144,7 +144,7 @@ int parse_opt_string_list(const struct option *opt, const char *arg, int unset) if (!arg) return -1; - string_list_append(v, xstrdup(arg)); + string_list_append(v, arg); return 0; } |