[v2] qapi: drop "must exist" from ID descriptions for consistency

[PATCH v2] qapi: drop "must exist" from ID descriptions for consistency

Filip Hejsek posted 1 patch 10 hours ago

Diff against v1
Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260609120221.461303-1-filip.hejsek@gmail.com

Maintainers: Eric Blake <eblake@redhat.com>, Markus Armbruster <armbru@redhat.com>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>

qapi/char.json | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

Expand all Fold all

[PATCH v2] qapi: drop "must exist" from ID descriptions for consistency

Posted by Filip Hejsek 10 hours ago

Make chardev ID param descriptions more consistent with ID descriptions
elsewhere.

Signed-off-by: Filip Hejsek <filip.hejsek@gmail.com>
---

On Tue, 2026-06-09 at 12:34 +0200, Markus Armbruster wrote:
> Uh, this fell through the cracks, possibly because it was buried within
> a thread.

It was fine, I was planning on rolling it into v7 of the resize series, but of
course you can merge it now if you want.

> Typo: consistent

On Tue, 2026-06-09 at 06:36 -0400, Michael S. Tsirkin wrote:
> maybe after it gets a signed off by tag from the author.

Both fixed.

 qapi/char.json | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/qapi/char.json b/qapi/char.json
index aa5ee9ffcd..abaccefa97 100644
--- a/qapi/char.json
+++ b/qapi/char.json
@@ -821,7 +821,7 @@
 #
 # Change a character device backend
 #
-# @id: the chardev's ID, must exist
+# @id: the chardev's ID
 #
 # @backend: new backend type and parameters
 #
@@ -862,7 +862,7 @@
 #
 # Remove a character device backend
 #
-# @id: the chardev's ID, must exist and not be in use
+# @id: the chardev's ID, must not be in use
 #
 # Since: 1.4
 #
@@ -879,7 +879,7 @@
 #
 # Send a break to a character device
 #
-# @id: the chardev's ID, must exist
+# @id: the chardev's ID
 #
 # Since: 2.10
 #
-- 
2.54.0