range-diff: publish default creation factor
authorEric Sunshine <sunshine@sunshineco.com>
Sun, 22 Jul 2018 09:57:11 +0000 (05:57 -0400)
committerJunio C Hamano <gitster@pobox.com>
Tue, 14 Aug 2018 21:24:56 +0000 (14:24 -0700)
The range-diff back-end allows its heuristic to be tweaked via the
"creation factor". git-range-diff, the only client of the back-end,
defaults the factor to 60% (hard-coded in builtin/range-diff.c), but
allows the user to override it with the --creation-factor option.

Publish the default range factor to allow new callers of the range-diff
back-end to default to the same value without duplicating the hard-coded
constant, and to avoid worrying about various callers becoming
out-of-sync if the default ever needs to change.

Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/range-diff.c
range-diff.h
index f52d45d9d614445e3473c810e9305c986a4dfdf8..006e76672631f679097d50f8db71f58cd7e677d6 100644 (file)
@@ -18,7 +18,7 @@ static struct strbuf *output_prefix_cb(struct diff_options *opt, void *data)
 
 int cmd_range_diff(int argc, const char **argv, const char *prefix)
 {
-       int creation_factor = 60;
+       int creation_factor = RANGE_DIFF_CREATION_FACTOR_DEFAULT;
        struct diff_options diffopt = { NULL };
        int simple_color = -1;
        struct option options[] = {
index 2407d46a301e5554bf3a4f855d20cc408b1f89bd..f26de5d0b04e8feefe044da6db63183f4e4e1fd8 100644 (file)
@@ -3,6 +3,8 @@
 
 #include "diff.h"
 
+#define RANGE_DIFF_CREATION_FACTOR_DEFAULT 60
+
 int show_range_diff(const char *range1, const char *range2,
                    int creation_factor, struct diff_options *diffopt);