Christian Couder <christian.couder@xxxxxxxxx> writes: > On Fri, Sep 29, 2017 at 8:49 AM, Pranit Bauva <pranit.bauva@xxxxxxxxx> wrote: >> Reimplement `is_expected_rev` & `check_expected_revs` shell function in >> C and add a `--check-expected-revs` subcommand to `git bisect--helper` to >> call it from git-bisect.sh . > > It looks like this patch forgot to add the `--check-expected-revs` > subcommand to the usage string. Usage string of what? "git bisect"? I do not think bisect--helper even needs to have end-user facing "usage string", but "git bisect--helper -h" still gets the new thing listed in its output for free thanks to parse-options API, so that is not what you found "this patch forgot to add" and you are making me scratch my head.