From: Markus Armbruster <armbru@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Kevin Wolf" <kwolf@redhat.com>,
"Daniel P . Berrangé" <berrange@redhat.com>,
"Eduardo Habkost" <ehabkost@redhat.com>,
qemu-block@nongnu.org, "Juan Quintela" <quintela@redhat.com>,
"Yuval Shaia" <yuval.shaia.ml@gmail.com>,
mdroth@linux.vnet.ibm.com,
"Dr . David Alan Gilbert" <dgilbert@redhat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Max Reitz" <mreitz@redhat.com>
Subject: [PATCH 00/20] Immutable QString, and also one JSON writer less
Date: Fri, 11 Dec 2020 18:11:32 +0100 [thread overview]
Message-ID: <20201211171152.146877-1-armbru@redhat.com> (raw)
Based-on: <20201210161452.2813491-1-armbru@redhat.com>
Cc: Daniel P. Berrangé <berrange@redhat.com>
Cc: Dr. David Alan Gilbert <dgilbert@redhat.com>
Cc: Eduardo Habkost <ehabkost@redhat.com>
Cc: Juan Quintela <quintela@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>
Cc: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
Cc: Max Reitz <mreitz@redhat.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Yuval Shaia <yuval.shaia.ml@gmail.com>
Cc: qemu-block@nongnu.org
Markus Armbruster (20):
hmp: Simplify how qmp_human_monitor_command() gets output
monitor: Use GString instead of QString for output buffer
qobject: Make qobject_to_json_pretty() take a pretty argument
qobject: Use GString instead of QString to accumulate JSON
qobject: Change qobject_to_json()'s value to GString
Revert "qstring: add qstring_free()"
hw/rdma: Replace QList by GQueue
qobject: Move internals to qobject-internal.h
qmp: Fix tracing of non-string command IDs
block: Avoid qobject_get_try_str()
Revert "qobject: let object_property_get_str() use new API"
qobject: Drop qobject_get_try_str()
qobject: Drop qstring_get_try_str()
qobject: Factor quoted_str() out of to_json()
qobject: Factor JSON writer out of qobject_to_json()
migration: Replace migration's JSON writer by the general one
json: Use GString instead of QString to accumulate strings
keyval: Use GString to accumulate value strings
block: Use GString instead of QString to build filenames
qobject: Make QString immutable
hw/rdma/rdma_backend_defs.h | 2 +-
hw/rdma/rdma_utils.h | 15 +-
include/migration/vmstate.h | 7 +-
include/qapi/qmp/json-writer.h | 35 ++++
include/qapi/qmp/qbool.h | 2 -
include/qapi/qmp/qdict.h | 2 -
include/qapi/qmp/qjson.h | 4 +-
include/qapi/qmp/qlist.h | 2 -
include/qapi/qmp/qnull.h | 2 -
include/qapi/qmp/qnum.h | 3 -
include/qapi/qmp/qobject.h | 9 +-
include/qapi/qmp/qstring.h | 14 +-
include/qemu/typedefs.h | 4 +-
migration/qjson.h | 29 ----
monitor/monitor-internal.h | 2 +-
qobject/qobject-internal.h | 39 +++++
block.c | 23 +--
block/rbd.c | 2 +-
hw/display/virtio-gpu.c | 2 +-
hw/intc/s390_flic_kvm.c | 2 +-
hw/nvram/eeprom93xx.c | 2 +-
hw/nvram/fw_cfg.c | 2 +-
hw/pci/msix.c | 2 +-
hw/pci/pci.c | 4 +-
hw/pci/shpc.c | 2 +-
hw/rdma/rdma_backend.c | 10 +-
hw/rdma/rdma_utils.c | 29 ++--
hw/rtc/twl92230.c | 2 +-
hw/scsi/scsi-bus.c | 2 +-
hw/usb/redirect.c | 7 +-
hw/virtio/virtio.c | 4 +-
migration/qjson.c | 114 -------------
migration/savevm.c | 53 ++++---
migration/vmstate-types.c | 38 ++---
migration/vmstate.c | 52 +++---
monitor/misc.c | 6 +-
monitor/monitor.c | 20 +--
monitor/qmp.c | 46 +++---
qemu-img.c | 33 ++--
qga/main.c | 22 +--
qobject/json-parser.c | 30 ++--
qobject/json-writer.c | 247 +++++++++++++++++++++++++++++
qobject/qbool.c | 1 +
qobject/qdict.c | 1 +
qobject/qjson.c | 144 ++++-------------
qobject/qlist.c | 1 +
qobject/qnull.c | 1 +
qobject/qnum.c | 6 +-
qobject/qobject.c | 1 +
qobject/qstring.c | 117 +++-----------
qom/object.c | 9 +-
qom/object_interfaces.c | 4 +-
qom/qom-hmp-cmds.c | 7 +-
target/alpha/machine.c | 2 +-
target/arm/machine.c | 6 +-
target/avr/machine.c | 4 +-
target/hppa/machine.c | 4 +-
target/microblaze/machine.c | 2 +-
target/mips/machine.c | 4 +-
target/openrisc/machine.c | 2 +-
target/ppc/machine.c | 10 +-
target/sparc/machine.c | 2 +-
tests/check-qjson.c | 67 ++++----
tests/check-qobject.c | 3 +-
tests/check-qstring.c | 16 --
tests/qtest/libqtest.c | 20 ++-
tests/test-visitor-serialization.c | 6 +-
util/keyval.c | 11 +-
migration/meson.build | 1 -
qobject/meson.build | 5 +-
70 files changed, 679 insertions(+), 705 deletions(-)
create mode 100644 include/qapi/qmp/json-writer.h
delete mode 100644 migration/qjson.h
create mode 100644 qobject/qobject-internal.h
delete mode 100644 migration/qjson.c
create mode 100644 qobject/json-writer.c
--
2.26.2
next reply other threads:[~2020-12-11 18:05 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-11 17:11 Markus Armbruster [this message]
2020-12-11 17:11 ` [PATCH 01/20] hmp: Simplify how qmp_human_monitor_command() gets output Markus Armbruster
2020-12-16 18:53 ` Dr. David Alan Gilbert
2020-12-11 17:11 ` [PATCH 02/20] monitor: Use GString instead of QString for output buffer Markus Armbruster
2020-12-16 19:36 ` Dr. David Alan Gilbert
2020-12-11 17:11 ` [PATCH 03/20] qobject: Make qobject_to_json_pretty() take a pretty argument Markus Armbruster
2020-12-11 17:11 ` [PATCH 04/20] qobject: Use GString instead of QString to accumulate JSON Markus Armbruster
2020-12-11 17:11 ` [PATCH 05/20] qobject: Change qobject_to_json()'s value to GString Markus Armbruster
2021-03-24 7:16 ` Thomas Huth
2021-03-24 8:14 ` Markus Armbruster
2020-12-11 17:11 ` [PATCH 06/20] Revert "qstring: add qstring_free()" Markus Armbruster
2020-12-11 17:11 ` [PATCH 07/20] hw/rdma: Replace QList by GQueue Markus Armbruster
2020-12-11 17:11 ` [PATCH 08/20] qobject: Move internals to qobject-internal.h Markus Armbruster
2020-12-11 17:11 ` [PATCH 09/20] qmp: Fix tracing of non-string command IDs Markus Armbruster
2020-12-11 17:11 ` [PATCH 10/20] block: Avoid qobject_get_try_str() Markus Armbruster
2020-12-11 18:28 ` Vladimir Sementsov-Ogievskiy
2020-12-11 17:11 ` [PATCH 11/20] Revert "qobject: let object_property_get_str() use new API" Markus Armbruster
2020-12-11 19:48 ` Eduardo Habkost
2020-12-11 17:11 ` [PATCH 12/20] qobject: Drop qobject_get_try_str() Markus Armbruster
2020-12-11 17:11 ` [PATCH 13/20] qobject: Drop qstring_get_try_str() Markus Armbruster
2020-12-11 17:11 ` [PATCH 14/20] qobject: Factor quoted_str() out of to_json() Markus Armbruster
2020-12-11 17:11 ` [PATCH 15/20] qobject: Factor JSON writer out of qobject_to_json() Markus Armbruster
2020-12-11 17:11 ` [PATCH 16/20] migration: Replace migration's JSON writer by the general one Markus Armbruster
2020-12-16 19:46 ` Dr. David Alan Gilbert
2020-12-17 7:10 ` Markus Armbruster
2020-12-17 9:38 ` Dr. David Alan Gilbert
2020-12-17 10:32 ` Dr. David Alan Gilbert
2020-12-11 17:11 ` [PATCH 17/20] json: Use GString instead of QString to accumulate strings Markus Armbruster
2020-12-11 17:11 ` [PATCH 18/20] keyval: Use GString to accumulate value strings Markus Armbruster
2020-12-22 9:56 ` Paolo Bonzini
2021-01-11 13:05 ` Markus Armbruster
2021-01-11 13:51 ` Paolo Bonzini
2020-12-11 17:11 ` [PATCH 19/20] block: Use GString instead of QString to build filenames Markus Armbruster
2020-12-11 18:47 ` Vladimir Sementsov-Ogievskiy
2020-12-11 17:11 ` [PATCH 20/20] qobject: Make QString immutable Markus Armbruster
2020-12-22 9:59 ` [PATCH 00/20] Immutable QString, and also one JSON writer less Paolo Bonzini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201211171152.146877-1-armbru@redhat.com \
--to=armbru@redhat.com \
--cc=berrange@redhat.com \
--cc=dgilbert@redhat.com \
--cc=ehabkost@redhat.com \
--cc=kwolf@redhat.com \
--cc=mdroth@linux.vnet.ibm.com \
--cc=mreitz@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
--cc=yuval.shaia.ml@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).