diff options
| author | Martijn Dekker <martijn@inlv.org> | 2019-06-29 00:17:42 +0200 |
|---|---|---|
| committer | Peter Stephenson <p.stephenson@samsung.com> | 2019-07-01 09:42:22 +0100 |
| commit | 700ec49581650ea1f0bffacb207a30145e278417 (patch) | |
| tree | b5fb684dd667ee80c7b42fee878ed1aae3645d52 /Src | |
| parent | 44446: Fix here document with file descriptor declarator. (diff) | |
| download | zsh-700ec49581650ea1f0bffacb207a30145e278417.tar zsh-700ec49581650ea1f0bffacb207a30145e278417.tar.gz zsh-700ec49581650ea1f0bffacb207a30145e278417.tar.bz2 zsh-700ec49581650ea1f0bffacb207a30145e278417.tar.lz zsh-700ec49581650ea1f0bffacb207a30145e278417.tar.xz zsh-700ec49581650ea1f0bffacb207a30145e278417.tar.zst zsh-700ec49581650ea1f0bffacb207a30145e278417.zip | |
44469: correct error on missing option argument
Diffstat (limited to 'Src')
| -rw-r--r-- | Src/builtin.c | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/Src/builtin.c b/Src/builtin.c index e863cc4bb..9b9e76c77 100644 --- a/Src/builtin.c +++ b/Src/builtin.c @@ -5511,14 +5511,12 @@ bin_getopts(UNUSED(char *name), char **argv, UNUSED(Options ops), UNUSED(int fun /* check for legality */ if(opch == ':' || !(p = memchr(optstr, opch, lenoptstr))) { p = "?"; - err: zsfree(zoptarg); setsparam(var, ztrdup(p)); if(quiet) { zoptarg = metafy(optbuf, lenoptbuf, META_DUP); } else { - zwarn(*p == '?' ? "bad option: %c%c" : - "argument expected after %c%c option", + zwarn("bad option: %c%c", "?-+"[lenoptbuf], opch); zoptarg=ztrdup(""); } @@ -5529,8 +5527,17 @@ bin_getopts(UNUSED(char *name), char **argv, UNUSED(Options ops), UNUSED(int fun if(p[1] == ':') { if(optcind == lenstr) { if(!args[zoptind]) { - p = ":"; - goto err; + zsfree(zoptarg); + if(quiet) { + setsparam(var, ztrdup(":")); + zoptarg = metafy(optbuf, lenoptbuf, META_DUP); + } else { + setsparam(var, ztrdup("?")); + zoptarg = ztrdup(""); + zwarn("argument expected after %c%c option", + "?-+"[lenoptbuf], opch); + } + return 0; } p = ztrdup(args[zoptind++]); } else |