Re: [RFC PATCH v2 1/4] builtin/rebase.c: reuse loop variable
From: Junio C Hamano <hidden>
Date: 2019-11-21 01:30:22
Andrei Rybak [off-list ref] writes:
Variable "int i" is already defined at the top of the function cmd_rebase, so reuse it instead of declaring other variables, which mask the outer "i".
The log message must also mention the other reason why this simplification is correct, namely, that outer "i" is dead at the point in the code that is touched by this patch and the value is never used in the later parts of the code (I just followed the codepath and made sure that is the case---iow, I think this patch is correct). Thanks.
quoted hunk
Signed-off-by: Andrei Rybak <redacted> --- builtin/rebase.c | 4 ---- 1 file changed, 4 deletions(-)diff --git a/builtin/rebase.c b/builtin/rebase.c index 4a20582e72..793cac1386 100644 --- a/builtin/rebase.c +++ b/builtin/rebase.c@@ -1747,8 +1747,6 @@ int cmd_rebase(int argc, const char **argv, const char *prefix) } if (exec.nr) { - int i; - imply_interactive(&options, "--exec"); strbuf_reset(&buf);@@ -1769,8 +1767,6 @@ int cmd_rebase(int argc, const char **argv, const char *prefix) } if (strategy_options.nr) { - int i; - if (!options.strategy) options.strategy = "recursive";