| commit | cd50a2564560986e865ff64fa73b59d2564076f0 | |
| author | Eric Blake <eblake@redhat.com> | |
| Thu, 14 Jul 2016 03:50:14 +0000 (13 21:50 -0600) | ||
| committer | Markus Armbruster <armbru@redhat.com> | |
| Tue, 19 Jul 2016 11:21:08 +0000 (19 13:21 +0200) | ||
| tree | e2e43efe9482f495760aff3b968fc69a817ddb0c | treesnapshot (tar.gz zip) |
| parent | d0b182392d0281ef780e3effcb82677a004f1f97 | commitdiff |
qapi: Special case c_name() for empty type
Commit 7ce106a rendered QAPISchemaObjectType.c_name() redundant,
since it now does nothing more than delegate to its superclass.
However, rather than deleting it, we can restore part of the
assertion that was removed in that commit, to prove that we never
emit the empty type directly in generated code, but rather
special-case it as a built-in that makes other aspects of code
generation easier to reason about.
Reported-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <1468468228-27827-4-git-send-email-eblake@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Commit 7ce106a rendered QAPISchemaObjectType.c_name() redundant,
since it now does nothing more than delegate to its superclass.
However, rather than deleting it, we can restore part of the
assertion that was removed in that commit, to prove that we never
emit the empty type directly in generated code, but rather
special-case it as a built-in that makes other aspects of code
generation easier to reason about.
Reported-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <1468468228-27827-4-git-send-email-eblake@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
| scripts/qapi.py | diffblobblamehistory |