From ae15fd4116212d8f2168e321594ee3acc2f50a5f Mon Sep 17 00:00:00 2001 From: Elijah Newren Date: Tue, 23 Aug 2022 02:42:21 +0000 Subject: [PATCH] merge: small code readability improvement After our loop through the selected strategies, we compare best_strategy to wt_strategy. This is fine, but the fact that the code setting best_strategy sets it to use_strategies[i]->name requires a little bit of extra checking to determine that at the time of setting, that's the same as wt_strategy. Just setting best_strategy to wt_strategy makes it a little easier to verify what the loop is doing, at least for this reader. Further, use_strategies[i]->name is used in a number of places, where we could just use wt_strategy. The latter takes less time for this reader to parse (one variable name instead of three), so just use wt_strategy to make the code slightly faster for human readers to parse. Signed-off-by: Elijah Newren Signed-off-by: Junio C Hamano --- builtin/merge.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/builtin/merge.c b/builtin/merge.c index 3e401de5cb..78af390ba3 100644 --- a/builtin/merge.c +++ b/builtin/merge.c @@ -1707,7 +1707,7 @@ int cmd_merge(int argc, const char **argv, const char *prefix) */ wt_strategy = use_strategies[i]->name; - ret = try_merge_strategy(use_strategies[i]->name, + ret = try_merge_strategy(wt_strategy, common, remoteheads, head_commit); /* @@ -1722,12 +1722,12 @@ int cmd_merge(int argc, const char **argv, const char *prefix) * another. */ merge_was_ok = 1; - best_strategy = use_strategies[i]->name; + best_strategy = wt_strategy; break; } cnt = (use_strategies_nr > 1) ? evaluate_result() : 0; if (best_cnt <= 0 || cnt <= best_cnt) { - best_strategy = use_strategies[i]->name; + best_strategy = wt_strategy; best_cnt = cnt; } }