diff options
| author | Eric Blake <eblake@redhat.com> | 2015-09-29 16:21:13 -0600 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2015-10-12 18:46:50 +0200 |
| commit | 1f35334489a43800df4d20cd91362a87cee39a29 (patch) | |
| tree | 8cb5cea8f18e493525e5a8908e109b705b6b1d6e /scripts/qapi.py | |
| parent | 05372f708a8cb3556e4d67458de79417dadf241f (diff) | |
| download | focaccia-qemu-1f35334489a43800df4d20cd91362a87cee39a29.tar.gz focaccia-qemu-1f35334489a43800df4d20cd91362a87cee39a29.zip | |
qapi: Share gen_err_check()
qapi-commands has a nice helper gen_err_check(), but did not use it everywhere. In fact, using it in more places makes it easier to reduce the lines of code used for generating error checks. This in turn will make it easier for later patches to consolidate another common pattern among the generators. The generated code has fewer blank lines in qapi-event.c functions, but has no semantic difference. Signed-off-by: Eric Blake <eblake@redhat.com> Message-Id: <1443565276-4535-16-git-send-email-eblake@redhat.com> [Drop another blank line for symmetry] Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'scripts/qapi.py')
| -rw-r--r-- | scripts/qapi.py | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/scripts/qapi.py b/scripts/qapi.py index c0728d73e1..62a415ccd9 100644 --- a/scripts/qapi.py +++ b/scripts/qapi.py @@ -1536,6 +1536,18 @@ def gen_params(arg_type, extra): ret += sep + extra return ret + +def gen_err_check(err='err', label='out'): + if not err: + return '' + return mcgen(''' + if (%(err)s) { + goto %(label)s; + } +''', + err=err, label=label) + + # # Common command line parsing # |