merge-recursive: check GIT_MERGE_VERBOSITY only once
authorAndrey Okoshkin <a.okoshkin@samsung.com>
Tue, 31 Oct 2017 09:09:13 +0000 (12:09 +0300)
committerJunio C Hamano <gitster@pobox.com>
Wed, 1 Nov 2017 01:11:56 +0000 (10:11 +0900)
Get rid of the duplicated getenv('GIT_MERGE_VERBOSITY') calls with the same
constant string argument. This makes code more readable and prevents typo in
the further development.

Signed-off-by: Andrey Okoshkin <a.okoshkin@samsung.com>
Reviewed-by: Stefan Beller <sbeller@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
merge-recursive.c
index 7a7d55aabeaf46bbd7f77b923e20a66ac4811d5e..6b3109e3a44d837fd067629a0d351784d3c60a89 100644 (file)
@@ -2162,6 +2162,7 @@ static void merge_recursive_config(struct merge_options *o)
 
 void init_merge_options(struct merge_options *o)
 {
+       const char *merge_verbosity;
        memset(o, 0, sizeof(struct merge_options));
        o->verbosity = 2;
        o->buffer_output = 1;
@@ -2170,9 +2171,9 @@ void init_merge_options(struct merge_options *o)
        o->renormalize = 0;
        o->detect_rename = 1;
        merge_recursive_config(o);
-       if (getenv("GIT_MERGE_VERBOSITY"))
-               o->verbosity =
-                       strtol(getenv("GIT_MERGE_VERBOSITY"), NULL, 10);
+       merge_verbosity = getenv("GIT_MERGE_VERBOSITY");
+       if (merge_verbosity)
+               o->verbosity = strtol(merge_verbosity, NULL, 10);
        if (o->verbosity >= 5)
                o->buffer_output = 0;
        strbuf_init(&o->obuf, 0);