On 9/19/18 2:09 PM, John Snow wrote:
We named these using underscores instead of the preferred dash,
document this nearby so we cannot possibly forget to rectify this
when we remove the 'x-' prefixes when the feature becomes stable.
We do not implement the change ahead of time to avoid more work
for libvirt to do in order to figure out how to use the beta version
of the API needlessly.
Indeed, deferring the spelling change until the name switch from 'x-' to
stable is an easily visible witness (via 'query-commands'), but
prematurely changing the name of the parameters for two different
versions of qemu while keeping the command named 'x-' (if we don't think
the feature is ready to promote to stable in for 3.1) would mean
resorting to the more computationally-expensive 'query-qmp-schema'.
Reported-by: Eric Blake <ebl...@redhat.com>
Signed-off-by: John Snow <js...@redhat.com>
---
qapi/block-core.json | 2 ++
1 file changed, 2 insertions(+)
Reviewed-by: Eric Blake <ebl...@redhat.com>
diff --git a/qapi/block-core.json b/qapi/block-core.json
index 4c7a37afdc..3611063073 100644
--- a/qapi/block-core.json
+++ b/qapi/block-core.json
@@ -1894,6 +1894,8 @@
##
# @x-block-dirty-bitmap-merge:
#
+# FIXME: Rename @src_name and @dst_name to src-name and dst-name.
+#
# Merge @src_name dirty bitmap to @dst_name dirty bitmap. @src_name dirty
# bitmap is unchanged. On error, @dst_name is unchanged.
#
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
