[PATCH v8 1/6] bisect--helper: BUG() in cmd_*() on invalid subcommand

From: Miriam Rubio <mirucam@gmail.com>
Date: Sep 23, 2020, 07:27 UTC
Message-ID: <20200923072740.20772-2-mirucam@gmail.com>
In-Reply-To: <20200923072740.20772-1-mirucam@gmail.com>

In cmd_bisect__helper() function, if an invalid or no subcommand is passed there is a BUG.

BUG() out instead of returning an error.

Mentored-by: Christian Couder <chriscool@tuxfamily.org>
Mentored-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Signed-off-by: Miriam Rubio <mirucam@gmail.com>
---
 builtin/bisect--helper.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/builtin/bisect--helper.c b/builtin/bisect--helper.c
index 7dcc1b5188..23a9f00035 100644
--- a/builtin/bisect--helper.c
+++ b/builtin/bisect--helper.c
@@ -720,7 +720,7 @@ int cmd_bisect__helper(int argc, const char **argv, const char *prefix)
 		res = bisect_start(&terms, argv, argc);
 		break;
 	default:
-		return error("BUG: unknown subcommand '%d'", cmdmode);
+		BUG("unknown subcommand %d", cmdmode);
 	}
 	free_terms(&terms);

-- 
2.25.0