diff options
| author | Markus Armbruster <armbru@redhat.com> | 2024-02-16 15:58:39 +0100 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2024-02-26 10:43:56 +0100 |
| commit | 629c5075aa6fb853855256cd7d380903e9b7ffbc (patch) | |
| tree | dd3f55dd787f40100652cecee3ff3a5312de0bbf /tests/qapi-schema/doc-duplicate-features.err | |
| parent | 3d035cd2cca66488f6f478a93b231c302466116b (diff) | |
| download | focaccia-qemu-629c5075aa6fb853855256cd7d380903e9b7ffbc.tar.gz focaccia-qemu-629c5075aa6fb853855256cd7d380903e9b7ffbc.zip | |
qapi: Reject multiple and empty feature descriptions
The parser recognizes only the first "Features:" line. Any subsequent ones are treated as ordinary text, as visible in test case doc-duplicate-features. Recognize "Features:" lines anywhere. A second one is an error. A 'Features:' line without any features is useless, but not an error. Make it an error. This makes detecting a second "Features:" line easier. qapi/run-state.json actually has an instance of this since commit fe17522d854 (qapi: Remove deprecated 'singlestep' member of StatusInfo). Clean it up. Signed-off-by: Markus Armbruster <armbru@redhat.com> Message-ID: <20240216145841.2099240-16-armbru@redhat.com> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Diffstat (limited to 'tests/qapi-schema/doc-duplicate-features.err')
| -rw-r--r-- | tests/qapi-schema/doc-duplicate-features.err | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/qapi-schema/doc-duplicate-features.err b/tests/qapi-schema/doc-duplicate-features.err index e69de29bb2..cadb2957a6 100644 --- a/tests/qapi-schema/doc-duplicate-features.err +++ b/tests/qapi-schema/doc-duplicate-features.err @@ -0,0 +1 @@ +doc-duplicate-features.json:9:1: duplicated 'Features:' line |