[PATCH v14 27/27] bisect--helper: remove the dequote in bisect_start()
From: Pranit Bauva <hidden>
Date: 2016-08-23 12:01:16
Subsystem:
the rest · Maintainer:
Linus Torvalds
Dequoting the arguments was introduced in 25b48b5c to port the function `bisect_next()` but after the `bisect_replay()` porting, the dequoting is carried out itself when it passes the arguments to `bisect_start()` in a simpler way thus dequoting again isn't required. So remove the extra "deqouting" code introduced by the commit 25b48b5c. Mentored-by: Lars Schneider [off-list ref] Mentored-by: Christian Couder [off-list ref] Signed-off-by: Pranit Bauva <redacted> --- builtin/bisect--helper.c | 59 ++++++++++++++++-------------------------------- 1 file changed, 20 insertions(+), 39 deletions(-)
diff --git a/builtin/bisect--helper.c b/builtin/bisect--helper.c
index 7577b69e..8bf495c 100644
--- a/builtin/bisect--helper.c
+++ b/builtin/bisect--helper.c@@ -541,67 +541,48 @@ static int bisect_start(struct bisect_terms *terms, int no_checkout, no_checkout = 1; for (i = 0; i < argc; i++) { - const char *arg; - if (starts_with(argv[i], "'")) - arg = sq_dequote(xstrdup(argv[i])); - else - arg = argv[i]; - if (!strcmp(arg, "--")) { + if (!strcmp(argv[i], "--")) { has_double_dash = 1; break; } } for (i = 0; i < argc; i++) { - const char *arg, *commit_id; - if (starts_with(argv[i], "'")) - arg = sq_dequote(xstrdup(argv[i])); - else - arg = argv[i]; - commit_id = xstrfmt("%s^{commit}", arg); + const char *commit_id; + commit_id = xstrfmt("%s^{commit}", argv[i]); if (!strcmp(argv[i], "--")) { has_double_dash = 1; break; - } else if (!strcmp(arg, "--no-checkout")) + } else if (!strcmp(argv[i], "--no-checkout")) no_checkout = 1; - else if (!strcmp(arg, "--term-good") || - !strcmp(arg, "--term-old")) { - const char *next_arg; - if (starts_with(argv[++i], "'")) - next_arg = sq_dequote(xstrdup(argv[i])); - else - next_arg = argv[i]; + else if (!strcmp(argv[i], "--term-good") || + !strcmp(argv[i], "--term-old")) { must_write_terms = 1; strbuf_reset(&terms->term_good); - strbuf_addstr(&terms->term_good, next_arg); - } else if (skip_prefix(arg, "--term-good=", &arg)) { + strbuf_addstr(&terms->term_good, argv[++i]); + } else if (skip_prefix(argv[i], "--term-good=", &argv[i])) { must_write_terms = 1; strbuf_reset(&terms->term_good); - strbuf_addstr(&terms->term_good, arg); - } else if (skip_prefix(arg, "--term-old=", &arg)) { + strbuf_addstr(&terms->term_good, argv[i]); + } else if (skip_prefix(argv[i], "--term-old=", &argv[i])) { must_write_terms = 1; strbuf_reset(&terms->term_good); - strbuf_addstr(&terms->term_good, arg); - } else if (!strcmp(arg, "--term-bad") || - !strcmp(arg, "--term-new")) { - const char *next_arg; - if (starts_with(argv[++i], "'")) - next_arg = sq_dequote(xstrdup(argv[i])); - else - next_arg = argv[i]; + strbuf_addstr(&terms->term_good, argv[i]); + } else if (!strcmp(argv[i], "--term-bad") || + !strcmp(argv[i], "--term-new")) { must_write_terms = 1; strbuf_reset(&terms->term_bad); - strbuf_addstr(&terms->term_bad, next_arg); - } else if (skip_prefix(arg, "--term-bad=", &arg)) { + strbuf_addstr(&terms->term_bad, argv[++i]); + } else if (skip_prefix(argv[i], "--term-bad=", &argv[i])) { must_write_terms = 1; strbuf_reset(&terms->term_bad); - strbuf_addstr(&terms->term_bad, arg); - } else if (skip_prefix(arg, "--term-new=", &arg)) { + strbuf_addstr(&terms->term_bad, argv[i]); + } else if (skip_prefix(argv[i], "--term-new=", &argv[i])) { must_write_terms = 1; strbuf_reset(&terms->term_good); - strbuf_addstr(&terms->term_good, arg); - } else if (starts_with(arg, "--") && - !one_of(arg, "--term-good", "--term-bad", NULL)) { + strbuf_addstr(&terms->term_good, argv[i]); + } else if (starts_with(argv[i], "--") && + !one_of(argv[i], "--term-good", "--term-bad", NULL)) { string_list_clear(&revs, 0); string_list_clear(&states, 0); die(_("unrecognised option: '%s'"), argv[i]); --
https://github.com/git/git/pull/287