has_help_option() uses its own parser. It's inconsistent with qemu_opts_parse(), as demonstrated by test-qemu-opts case /qemu-opts/has_help_option. Fix by reusing the common parser.
Signed-off-by: Markus Armbruster <[email protected]> --- tests/test-qemu-opts.c | 2 +- util/qemu-option.c | 39 +++++++++++++++++++-------------------- 2 files changed, 20 insertions(+), 21 deletions(-) diff --git a/tests/test-qemu-opts.c b/tests/test-qemu-opts.c index 27c24bb1a2..58a4ea2408 100644 --- a/tests/test-qemu-opts.c +++ b/tests/test-qemu-opts.c @@ -744,7 +744,7 @@ static void test_has_help_option(void) { "a,help", true, true, true }, { "a=0,help,b", true, true, true }, { "help,b=1", true, true, false }, - { "a,b,,help", false /* BUG */, true, true }, + { "a,b,,help", true, true, true }, }; int i; QemuOpts *opts; diff --git a/util/qemu-option.c b/util/qemu-option.c index 6403e521fc..279f1b3fb3 100644 --- a/util/qemu-option.c +++ b/util/qemu-option.c @@ -165,26 +165,6 @@ void parse_option_size(const char *name, const char *value, *ret = size; } -bool has_help_option(const char *param) -{ - const char *p = param; - bool result = false; - - while (*p && !result) { - char *value; - - p = get_opt_value(p, &value); - if (*p) { - p++; - } - - result = is_help_option(value); - g_free(value); - } - - return result; -} - bool is_valid_option_list(const char *p) { char *value = NULL; @@ -890,6 +870,25 @@ static char *opts_parse_id(const char *params) return NULL; } +bool has_help_option(const char *params) +{ + const char *p; + char *name, *value; + bool ret; + + for (p = params; *p;) { + p = get_opt_name_value(p, NULL, &name, &value); + ret = !strcmp(name, "help"); + g_free(name); + g_free(value); + if (ret) { + return true; + } + } + + return false; +} + /** * Store options parsed from @params into @opts. * If @firstname is non-null, the first key=value in @params may omit -- 2.21.1
