[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 07/10] qemu-storage-daemon: Remove QemuOpts from --object parser
From: |
Kevin Wolf |
Subject: |
[PULL 07/10] qemu-storage-daemon: Remove QemuOpts from --object parser |
Date: |
Thu, 15 Oct 2020 16:49:49 +0200 |
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>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20201007164903.282198-5-kwolf@redhat.com>
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 6f0e0cfb36..e419ba9f19 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;
--
2.28.0
- [PULL 00/10] Block layer patches, Kevin Wolf, 2020/10/15
- [PULL 01/10] keyval: Fix and clarify grammar, Kevin Wolf, 2020/10/15
- [PULL 02/10] test-keyval: Demonstrate misparse of ', ' with implied key, Kevin Wolf, 2020/10/15
- [PULL 03/10] keyval: Fix parsing of ',' in value of implied key, Kevin Wolf, 2020/10/15
- [PULL 05/10] qom: Factor out helpers from user_creatable_print_help(), Kevin Wolf, 2020/10/15
- [PULL 06/10] qom: Add user_creatable_print_help_from_qdict(), Kevin Wolf, 2020/10/15
- [PULL 04/10] keyval: Parse help options, Kevin Wolf, 2020/10/15
- [PULL 09/10] block: drop moderated sheepdog mailing list from MAINTAINERS file, Kevin Wolf, 2020/10/15
- [PULL 07/10] qemu-storage-daemon: Remove QemuOpts from --object parser,
Kevin Wolf <=
- [PULL 08/10] monitor: Fix order in monitor_cleanup(), Kevin Wolf, 2020/10/15
- [PULL 10/10] block: deprecate the sheepdog block driver, Kevin Wolf, 2020/10/15
- Re: [PULL 00/10] Block layer patches, Peter Maydell, 2020/10/16