Subject: Re: [PATCH v5 01/13] bisect--helper: BUG() in cmd_*() on invalid subcommand

From: Shourya Shukla <[email-masked]>
Date: Sat, 25 Jul 2020 22:57:56 +0530

Hello Miriam,

> In cmd_bisect__helper() function, there would be a bug
> if an invalid or no subcommand was passed.return error()
> should be replaced by BUG() in that case.

> Mentored-by: Christian Couder <[email-masked]>
> Mentored-by: Johannes Schindelin <[email-masked]>
> Signed-off-by: Miriam Rubio <[email-masked]>
> ---

I think we can try for a better message. Maybe something like:

    One encounters a BUG when an invalid or no subcommand is passed to
    'cmd_bisect__helper()'. BUG() out instead of returning an error.

> -		return error("BUG: unknown subcommand '%d'", cmdmode);
> +		BUG("unknown subcommand %d", (int)cmdmode);

Do we need to typecast an integer to an integer here?

Regards,
Shourya Shukla