[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/13] qdict: Make qdict_flatten() shallow-clone
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 09/13] qdict: Make qdict_flatten() shallow-clone-friendly |
Date: |
Fri, 11 May 2018 13:44:01 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 05/09/2018 11:55 AM, Max Reitz wrote:
In its current form, qdict_flatten() removes all entries from nested
QDicts that are moved to the root QDict. It is completely sufficient to
remove all old entries from the root QDict, however. If the nested
dicts have a refcount of 1, this will automatically delete them, too.
And if they have a greater refcount, we probably do not want to modify
them in the first place.
The latter observation means that it was currently (in general)
impossible to qdict_flatten() a shallowly cloned dict because that would
empty nested QDicts in the original dict as well. This patch changes
this, so you can now use qdict_flatten(qdict_shallow_clone(dict)) to get
a flattened copy without disturbing the original.
Signed-off-by: Max Reitz <address@hidden>
---
qobject/qdict.c | 23 ++++++++++++++++-------
1 file changed, 16 insertions(+), 7 deletions(-)
g_free(new_key);
- if (delete) {
+ if (copied_value && qdict == target) {
+ /* If we have copied a value, and if we are on the root
+ * level, we need to remove the old entry. Otherwise, we
+ * do not, because by removing these entries on the root
+ * level, the reference counts of nested dicts and listed
s/listed/lists/
+ * will be reduced automatically. In fact, we probably do
+ * not want to modify nested dicts and lists with
+ * refcounts greater than 1 anyway. */
qdict_del(qdict, entry->key);
- /* Restart loop after modifying the iterated QDict */
+ /* Restart loop after modifying the iterated QDict. We
+ * only need to do this if qdict == target, because
+ * otherwise copying the value did not affect qdict. */
entry = qdict_first(qdict);
continue;
}
With the typo fix,
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
- Re: [Qemu-devel] [PATCH 07/13] qdict: Add qdict_stringify_for_keyval(), (continued)
- [Qemu-devel] [PATCH 06/13] block: Add block-specific QDict header, Max Reitz, 2018/05/09
- [Qemu-devel] [PATCH 09/13] qdict: Make qdict_flatten() shallow-clone-friendly, Max Reitz, 2018/05/09
- Re: [Qemu-devel] [PATCH 09/13] qdict: Make qdict_flatten() shallow-clone-friendly,
Eric Blake <=
- [Qemu-devel] [PATCH 10/13] tests: Add QDict clone-flatten test, Max Reitz, 2018/05/09
- [Qemu-devel] [PATCH 12/13] iotests: Test internal option typing, Max Reitz, 2018/05/09
- [Qemu-devel] [PATCH 11/13] block: Try to create well typed json:{} filenames, Max Reitz, 2018/05/09
- [Qemu-devel] [PATCH 13/13] iotests: qcow2's encrypt.format is now optional, Max Reitz, 2018/05/09