qapi: Move @String out of common.json to discourage reuse
authorMarkus Armbruster <armbru@redhat.com>
Mon, 5 Feb 2024 07:47:08 +0000 (08:47 +0100)
committerMarkus Armbruster <armbru@redhat.com>
Mon, 12 Feb 2024 09:04:32 +0000 (10:04 +0100)
Use of String is problematic, because it results in awkward interface
documentation.  The previous commit cleaned up one instance.

Move String out of common.json next to its remaining users in net.json
to discourage reuse elsewhere.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-ID: <20240205074709.3613229-15-armbru@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
include/net/filter.h
qapi/common.json
qapi/net.json

index 27ffc630df4225d85a00687aa1120f3100dee29a..f15f7932b2982c47b523c68cff6b561f15196b5b 100644 (file)
@@ -9,7 +9,7 @@
 #ifndef QEMU_NET_FILTER_H
 #define QEMU_NET_FILTER_H
 
-#include "qapi/qapi-types-net.h"
+#include "qapi/qapi-types-common.h"
 #include "qemu/queue.h"
 #include "qom/object.h"
 #include "net/queue.h"
index 6fed9cde1a9e11839d9532c72d5e6311c378d177..f1bb841951c4e007406d948d4a09fa3b5c003a1f 100644 (file)
 { 'enum': 'OnOffSplit',
   'data': [ 'on', 'off', 'split' ] }
 
-##
-# @String:
-#
-# A fat type wrapping 'str', to be embedded in lists.
-#
-# Since: 1.2
-##
-{ 'struct': 'String',
-  'data': {
-    'str': 'str' } }
-
 ##
 # @StrOrNull:
 #
index 68493d6ac9892d1c0baf78f6a8abeef9e88b3628..0a993e1a3d2ba0eb374541fe9f597afabc7eef64 100644 (file)
@@ -6,7 +6,6 @@
 # = Net devices
 ##
 
-{ 'include': 'common.json' }
 { 'include': 'sockets.json' }
 
 ##
     '*addr':    'str',
     '*vectors': 'uint32' } }
 
+##
+# @String:
+#
+# A fat type wrapping 'str', to be embedded in lists.
+#
+# Since: 1.2
+##
+{ 'struct': 'String',
+  'data': {
+    'str': 'str' } }
+
 ##
 # @NetdevUserOptions:
 #