diff options
| author | Markus Armbruster <armbru@redhat.com> | 2021-08-31 14:38:01 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2021-09-03 17:09:10 +0200 |
| commit | dd044023e6c1000e384c511e2d8f4c9d1a2a3e91 (patch) | |
| tree | b878e7fd1f5c7a2ee6010462ff31cda063335f34 /scripts/qapi/common.py | |
| parent | cdcc04fa035025e706fb55b2a9e4843a54177ae4 (diff) | |
| download | focaccia-qemu-dd044023e6c1000e384c511e2d8f4c9d1a2a3e91.tar.gz focaccia-qemu-dd044023e6c1000e384c511e2d8f4c9d1a2a3e91.zip | |
tests/qapi-schema: Demonstrate broken C code for 'if'
The C code generated for 'if' conditionals is incorrectly
parenthesized. For instance,
'if': { 'not': { 'any': [ { 'not': 'TEST_IF_EVT' },
{ 'not': 'TEST_IF_STRUCT' } ] } } }
generates
#if !(!defined(TEST_IF_EVT)) || (!defined(TEST_IF_STRUCT))
This is wrong. Correct would be:
#if !(!defined(TEST_IF_EVT) || !defined(TEST_IF_STRUCT))
Cover the issue in qapi-schema-test.json. This generates bad #if in
tests/test-qapi-events.h and other files.
Add a similar condition to doc-good.json. The generated documentation
is fine.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20210831123809.1107782-5-armbru@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Diffstat (limited to 'scripts/qapi/common.py')
0 files changed, 0 insertions, 0 deletions