[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 4/4] qemu-storage-daemon: Remove QemuOpts from --object pa
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v2 4/4] qemu-storage-daemon: Remove QemuOpts from --object parser |
Date: |
Fri, 02 Oct 2020 14:26:43 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Kevin Wolf <kwolf@redhat.com> writes:
> The command line parser for --object parses the input twice: Once into
> QemuOpts just for detecting help options, and then again into a QDict
> using the keyval parser for actually creating the object.
>
> Now that the keyval parser can also detect help options, we can simplify
> this and remove the QemuOpts part.
>
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
> ---
> storage-daemon/qemu-storage-daemon.c | 15 ++++-----------
> 1 file changed, 4 insertions(+), 11 deletions(-)
>
> diff --git a/storage-daemon/qemu-storage-daemon.c
> b/storage-daemon/qemu-storage-daemon.c
> index bb9cb740f0..7cbdbf0b23 100644
> --- a/storage-daemon/qemu-storage-daemon.c
> +++ b/storage-daemon/qemu-storage-daemon.c
> @@ -264,21 +264,14 @@ static void process_options(int argc, char *argv[])
> }
> case OPTION_OBJECT:
> {
> - QemuOpts *opts;
> - const char *type;
> QDict *args;
> + bool help;
>
> - /* FIXME The keyval parser rejects 'help' arguments, so we
> must
> - * unconditionall try QemuOpts first. */
> - opts = qemu_opts_parse(&qemu_object_opts,
> - optarg, true, &error_fatal);
> - type = qemu_opt_get(opts, "qom-type");
> - if (type && user_creatable_print_help(type, opts)) {
> + args = keyval_parse(optarg, "qom-type", &help, &error_fatal);
> + if (help) {
> + user_creatable_print_help_from_qdict(args);
> exit(EXIT_SUCCESS);
> }
> - qemu_opts_del(opts);
> -
> - args = keyval_parse(optarg, "qom-type", NULL, &error_fatal);
> user_creatable_add_dict(args, true, &error_fatal);
> qobject_unref(args);
> break;
Reviewed-by: Markus Armbruster <armbru@redhat.com>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH v2 4/4] qemu-storage-daemon: Remove QemuOpts from --object parser,
Markus Armbruster <=