Re: [PATCH 1/2] bisect--helper: use OPT_CMDMODE instead of OPT_BOOL

Eric Sunshine [off-list ref] writes:

quoted

+enum sub_commands {

How about calling this 'enum subcommand' (no underscore, non-plural)?

quoted

+       NEXT_ALL = 1
+};
+
 int cmd_bisect__helper(int argc, const char **argv, const char *prefix)
 {
-       int next_all = 0;
+       int sub_command = 0;

s/sub_command/subcommand/

Non-plural is a good suggestion, but remember that you are using
OPT_CMDMODE to parse out that thing.  There is already a name for
that concept: "command name".  Why not name these after that phrase,
just like merge-base, replace, and tag already do?