diff options
author | Nicholas Marriott <nicm@openbsd.org> | 2009-07-07 21:23:22 +0000 |
---|---|---|
committer | Nicholas Marriott <nicm@openbsd.org> | 2009-07-07 21:23:22 +0000 |
commit | b4efd1ca89132dcf24acd6dc7b1ccee369b7d90c (patch) | |
tree | d35e3a200addeafa5a1a5d578aa81655e3baddd4 | |
parent | 923ccfa2e8dfddc1611213bc027fd2968a84c04b (diff) | |
download | rtmux-b4efd1ca89132dcf24acd6dc7b1ccee369b7d90c.tar.gz rtmux-b4efd1ca89132dcf24acd6dc7b1ccee369b7d90c.tar.bz2 rtmux-b4efd1ca89132dcf24acd6dc7b1ccee369b7d90c.zip |
Don't let ambiguous commands override an exact alias match: eg if commands
"abc-1", "abc-2", "abc-3" exist and "abc-3" has the alias "abc", "tmux abc"
should execute abc-3, not complain about the command being ambiguous.
Not a problem at the moment but will be soon.
-rw-r--r-- | cmd.c | 7 |
1 files changed, 5 insertions, 2 deletions
@@ -105,7 +105,7 @@ cmd_parse(int argc, char **argv, char **cause) const struct cmd_entry **entryp, *entry; struct cmd *cmd; char s[BUFSIZ]; - int opt; + int opt, ambiguous = 0; *cause = NULL; if (argc == 0) { @@ -117,6 +117,7 @@ cmd_parse(int argc, char **argv, char **cause) for (entryp = cmd_table; *entryp != NULL; entryp++) { if ((*entryp)->alias != NULL && strcmp((*entryp)->alias, argv[0]) == 0) { + ambiguous = 0; entry = *entryp; break; } @@ -124,13 +125,15 @@ cmd_parse(int argc, char **argv, char **cause) if (strncmp((*entryp)->name, argv[0], strlen(argv[0])) != 0) continue; if (entry != NULL) - goto ambiguous; + ambiguous = 1; entry = *entryp; /* Bail now if an exact match. */ if (strcmp(entry->name, argv[0]) == 0) break; } + if (ambiguous) + goto ambiguous; if (entry == NULL) { xasprintf(cause, "unknown command: %s", argv[0]); return (NULL); |