From 77b946689ae949381954aa34e9ca863f54839abb Mon Sep 17 00:00:00 2001 From: Curtis McEnroe Date: Wed, 27 Feb 2019 14:33:11 -0500 Subject: Use getsubopt(3) to parse -o options in hi --- bin/hi.c | 22 ++++++++-------------- 1 file changed, 8 insertions(+), 14 deletions(-) diff --git a/bin/hi.c b/bin/hi.c index 68a50469..3c312c4a 100644 --- a/bin/hi.c +++ b/bin/hi.c @@ -349,10 +349,11 @@ enum Option { OptionLen, }; -static const char *OptionKey[OptionLen] = { +static const char *OptionKey[OptionLen + 1] = { #define X(option, key) [option] = key, ENUM_OPTION #undef X + NULL, }; typedef void HeaderFn(const char *opts[]); @@ -649,13 +650,6 @@ static bool findFormat(struct Format *format, const char *name) { return false; } -static bool findOption(enum Option *opt, const char *key) { - for (*opt = 0; *opt < OptionLen; ++*opt) { - if (!strcmp(key, OptionKey[*opt])) return true; - } - return false; -} - int main(int argc, char *argv[]) { setlocale(LC_CTYPE, ""); @@ -680,14 +674,14 @@ int main(int argc, char *argv[]) { } break; case 'n': name = optarg; break; case 'o': { + char *val; enum Option key; - char *keystr, *valstr; - while (NULL != (valstr = strsep(&optarg, ","))) { - keystr = strsep(&valstr, "="); - if (!findOption(&key, keystr)) { - errx(EX_USAGE, "no such option %s", keystr); + while (optarg[0]) { + key = getsubopt(&optarg, (char *const *)OptionKey, &val); + if (key >= OptionLen) { + errx(EX_USAGE, "no such option %s", suboptarg); } - opts[key] = (valstr ? valstr : keystr); + opts[key] = (val ? val : ""); } } break; default: return EX_USAGE; -- cgit 1.4.1