Re: [PATCH 02/16] tests/qapi-schema: Cover duplicate 'Features:' line

[Daniel P . Berrangé]

On Fri, Feb 16, 2024 at 03:58:26PM +0100, Markus Armbruster wrote:
> We don't actually recognize the second 'Features:' line.  Instead, we
> treat it as an untagged section.
> 
> If it was followed by feature description, we'd reject that like
> "description of '@feat2:' follows a section".  Less than clear.
> 
> To be improved shortly.
> 
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  tests/qapi-schema/doc-duplicate-features.err  |  0
>  tests/qapi-schema/doc-duplicate-features.json | 11 ++++++++++
>  tests/qapi-schema/doc-duplicate-features.out  | 22 +++++++++++++++++++
>  tests/qapi-schema/meson.build                 |  1 +
>  4 files changed, 34 insertions(+)
>  create mode 100644 tests/qapi-schema/doc-duplicate-features.err
>  create mode 100644 tests/qapi-schema/doc-duplicate-features.json
>  create mode 100644 tests/qapi-schema/doc-duplicate-features.out

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|