[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 17/24] qapi: New qobject_input_visitor_new_str()
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH 17/24] qapi: New qobject_input_visitor_new_str() for convenience |
Date: |
Tue, 28 Feb 2017 18:33:32 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 27.02.2017 um 12:20 hat Markus Armbruster geschrieben:
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
> include/qapi/qobject-input-visitor.h | 12 ++++++++++++
> qapi/qobject-input-visitor.c | 33 +++++++++++++++++++++++++++++++++
> 2 files changed, 45 insertions(+)
>
> diff --git a/include/qapi/qobject-input-visitor.h
> b/include/qapi/qobject-input-visitor.h
> index 282f9d2..b399285 100644
> --- a/include/qapi/qobject-input-visitor.h
> +++ b/include/qapi/qobject-input-visitor.h
> @@ -68,4 +68,16 @@ Visitor *qobject_input_visitor_new(QObject *obj);
> */
> Visitor *qobject_input_visitor_new_keyval(QObject *obj);
>
> +/*
> + * Create a QObject input visitor for parsing @str.
> + *
> + * If @str looks like JSON, parse it as JSON, else as KEY=VALUE,...
> + * @implied_key applies to KEY=VALUE, and works as in keyval_parse().
> + * On failure, store an error through @errp and return NULL.
> + * On success, return a new QObject input visitor for the parse.
> + */
> +Visitor *qobject_input_visitor_new_str(const char *str,
> + const char *implied_key,
> + Error **errp);
> +
> #endif
> diff --git a/qapi/qobject-input-visitor.c b/qapi/qobject-input-visitor.c
> index 3db5850..64a08d3 100644
> --- a/qapi/qobject-input-visitor.c
> +++ b/qapi/qobject-input-visitor.c
> @@ -18,9 +18,11 @@
> #include "qapi/visitor-impl.h"
> #include "qemu/queue.h"
> #include "qemu-common.h"
> +#include "qapi/qmp/qjson.h"
> #include "qapi/qmp/types.h"
> #include "qapi/qmp/qerror.h"
> #include "qemu/cutils.h"
> +#include "qemu/option.h"
>
> typedef struct StackObject {
> const char *name; /* Name of @obj in its parent, if any */
> @@ -655,3 +657,34 @@ Visitor *qobject_input_visitor_new_keyval(QObject *obj)
>
> return &v->visitor;
> }
> +
> +Visitor *qobject_input_visitor_new_str(const char *str,
> + const char *implied_key,
> + Error **errp)
> +{
> + bool is_json = str[0] == '{';
> + QObject *obj;
> + QDict *args;
> + Visitor *v;
> +
> + if (is_json) {
> + obj = qobject_from_json(str, errp);
> + if (!obj) {
> + /* Work around qobject_from_json() lossage TODO fix that */
> + if (errp && !*errp) {
> + error_setg(errp, "JSON parse error");
> + return NULL;
> + }
> + return NULL;
> + }
> + args = qobject_to_qdict(obj);
> + assert(args);
> + v = qobject_input_visitor_new(QOBJECT(args));
> + } else {
> + args = keyval_parse(optarg, implied_key, errp);
No, you don't mean optarg.
Makes all attempts to use -blockdev segfault for me.
> + v = qobject_input_visitor_new_keyval(QOBJECT(args));
> + }
> + QDECREF(args);
> +
> + return v;
> +}
Kevin
- [Qemu-devel] [PATCH 09/24] libqtest: Fix qmp() & friends to abort on JSON parse errors, (continued)
- [Qemu-devel] [PATCH 09/24] libqtest: Fix qmp() & friends to abort on JSON parse errors, Markus Armbruster, 2017/02/27
- [Qemu-devel] [PATCH 21/24] test-qapi-util: New, covering qapi/qapi-util.c, Markus Armbruster, 2017/02/27
- [Qemu-devel] [PATCH 18/24] block: Initial implementation of -blockdev, Markus Armbruster, 2017/02/27
- [Qemu-devel] [PATCH 17/24] qapi: New qobject_input_visitor_new_str() for convenience, Markus Armbruster, 2017/02/27
- Re: [Qemu-devel] [PATCH 17/24] qapi: New qobject_input_visitor_new_str() for convenience,
Kevin Wolf <=
- [Qemu-devel] [PATCH 02/24] tests: Fix gcov-files-test-qemu-opts-y, gcov-files-test-logging-y, Markus Armbruster, 2017/02/27
- [Qemu-devel] [PATCH 08/24] qobject: Propagate parse errors through qobject_from_jsonv(), Markus Armbruster, 2017/02/27
- [Qemu-devel] [PATCH 07/24] qapi: Factor out common qobject_input_get_keyval(), Markus Armbruster, 2017/02/27
- [Qemu-devel] [PATCH 23/24] keyval: Restrict key components to valid QAPI names, Markus Armbruster, 2017/02/27
- [Qemu-devel] [PATCH 19/24] qapi: Improve how keyval input visitor reports unexpected dicts, Markus Armbruster, 2017/02/27