Erik Faye-Lund <kusmabite@xxxxxxxxx> writes: > For this code-path to trigger we would have to be able to prefix-match > every common command AND every "main command" must be included in > common commands. At the same time. The only possible way to > prefix-match all commands is if they all start with the same letter. > Do you really think this is a situation we could ever end up in? Every > git command being a common-command, starting with the same letter? > > This is basically unreachable code. Perhaps it'd be even clearer just to die: > > if (main_cmds.cnt <= n) > die("Prefix-matched everyting, what's going on?"); Well, the same letter can be an empty string: $ git '' Didn't I already suggest BUG() there? Also, saying "too ambiguous" would make the codepath give "... See 'git --help'" message, I think. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html