diff options
| author | Markus Armbruster | 2020-12-11 18:11:37 +0100 |
|---|---|---|
| committer | Markus Armbruster | 2020-12-19 10:38:43 +0100 |
| commit | eab3a4678b07267c39e7290a6e9e7690b1d2a521 (patch) | |
| tree | 223d43bbde1e59f4f6867a36e2dfdf0ca36603f3 /qom/object_interfaces.c | |
| parent | qobject: Use GString instead of QString to accumulate JSON (diff) | |
| download | qemu-eab3a4678b07267c39e7290a6e9e7690b1d2a521.tar.gz qemu-eab3a4678b07267c39e7290a6e9e7690b1d2a521.tar.xz qemu-eab3a4678b07267c39e7290a6e9e7690b1d2a521.zip | |
qobject: Change qobject_to_json()'s value to GString
qobject_to_json() and qobject_to_json_pretty() build a GString, then
covert it to QString. Just one of the callers actually needs a
QString: qemu_rbd_parse_filename(). A few others need a string they
can modify: qmp_send_response(), qga's send_response(), to_json_str(),
and qmp_fd_vsend_fds(). The remainder just need a string.
Change qobject_to_json() and qobject_to_json_pretty() to return the
GString.
qemu_rbd_parse_filename() now has to convert to QString. All others
save a QString temporary. to_json_str() actually becomes a bit
simpler, because GString provides more convenient modification
functions.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20201211171152.146877-6-armbru@redhat.com>
Diffstat (limited to 'qom/object_interfaces.c')
| -rw-r--r-- | qom/object_interfaces.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/qom/object_interfaces.c b/qom/object_interfaces.c index ed896fe764..1e9ad6f08a 100644 --- a/qom/object_interfaces.c +++ b/qom/object_interfaces.c @@ -5,7 +5,6 @@ #include "qapi/qmp/qdict.h" #include "qapi/qmp/qerror.h" #include "qapi/qmp/qjson.h" -#include "qapi/qmp/qstring.h" #include "qapi/qobject-input-visitor.h" #include "qom/object_interfaces.h" #include "qemu/help_option.h" @@ -207,7 +206,8 @@ char *object_property_help(const char *name, const char *type, g_string_append(str, description); } if (defval) { - g_autofree char *def_json = qstring_free(qobject_to_json(defval), TRUE); + g_autofree char *def_json = g_string_free(qobject_to_json(defval), + true); g_string_append_printf(str, " (default: %s)", def_json); } |