On Fri, Nov 04, 2022 at 06:40:11PM +0700, Đoàn Trần Công Danh wrote: > In a later change, we will use OPT_SUBCOMMAND to parse sub-commands to > avoid consuming non-option opts. > > Since OPT_SUBCOMMAND needs a function pointer to operate, > let's move it now. Makes sense, and splitting this into its own step makes it easier to verify. > +static int cmd_bisect__reset(int argc, const char **argv, const char *prefix UNUSED) Thank you very much for putting the UNUSED markers here. It will save me the trouble of handling it on my topic. :) -Peff