[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH 17/24] qapi: New qobject_input_visi
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH 17/24] qapi: New qobject_input_visitor_new_str() for convenience |
Date: |
Tue, 28 Feb 2017 19:45:17 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Kevin Wolf <address@hidden> writes:
> 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.
Okay, I guess I get to stay after class and write "Late night last
minute changes are a bad idea" one hundred times.
Including the kitchen sink can hide stupid bugs...
>
>> + v = qobject_input_visitor_new_keyval(QOBJECT(args));
>> + }
>> + QDECREF(args);
>> +
>> + return v;
>> +}
>
> Kevin
Re: [Qemu-block] [PATCH 00/24] block: Command line option -blockdev, Eric Blake, 2017/02/28