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

[Markus Armbruster <armbru@redhat.com>]

"Michael S. Tsirkin" <mst@redhat.com> writes:

> On Tue, Jun 09, 2026 at 12:34:03PM +0200, Markus Armbruster wrote:
>> Uh, this fell through the cracks, possibly because it was buried within
>> a thread.
>> 
>> Filip Hejsek <filip.hejsek@gmail.com> writes:
>> 
>> > Make chardev ID param descriptions more consistend with ID descriptions
>> 
>> Typo: consistent
>> 
>> Not worth a respin.
>> 
>> > elsewhere.
>> > ---
>> >  qapi/char.json | 6 +++---
>> >  1 file changed, 3 insertions(+), 3 deletions(-)
>> >
>> > diff --git a/qapi/char.json b/qapi/char.json
>> > index 140614f82c..fbac5a1cb7 100644
>> > --- a/qapi/char.json
>> > +++ b/qapi/char.json
>> > @@ -791,7 +791,7 @@
>> >  #
>> >  # Change a character device backend
>> >  #
>> > -# @id: the chardev's ID, must exist
>> > +# @id: the chardev's ID
>> >  #
>> >  # @backend: new backend type and parameters
>> >  #
>> > @@ -832,7 +832,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
>> >  #
>> > @@ -849,7 +849,7 @@
>> >  #
>> >  # Send a break to a character device
>> >  #
>> > -# @id: the chardev's ID, must exist
>> > +# @id: the chardev's ID
>> >  #
>> >  # Since: 2.10
>> >  #
>> 
>> Reviewed-by: Markus Armbruster <armbru@redhat.com>
>> 
>> I can take this.
>
> maybe after it gets a signed off by tag from the author.

Oops, missed that!  Can't take it without.

Filip?