]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/qemu-xen.git/commit
projects / people / liuw / libxenctrl-split / qemu-xen.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 1ffe818) | patch
qapi: Improve 'include' error message
authorEric Blake <eblake@redhat.com>
Tue, 29 Sep 2015 22:21:00 +0000 (16:21 -0600)
committerMarkus Armbruster <armbru@redhat.com>
Mon, 12 Oct 2015 16:44:54 +0000 (18:44 +0200)
commit7408fb67c0f9403f6e40aecf97cf798fc14e2cd8
treea73da8f810d2e42c7058c65659c41c13fc6ce658tree
parent1ffe818a395cb883746f3baf8d9a0b6988375e8bcommit | diff
qapi: Improve 'include' error message

Use of '"...%s" % include' to print non-strings can lead to
ugly messages, such as this (if the .json change is applied
without the qapi.py change):
 Expected a file name (string), got: OrderedDict()

Better is to just omit the actual non-string value in the
message.

Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <1443565276-4535-3-git-send-email-eblake@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
scripts/qapi.py diff | blob | blame | history
tests/qapi-schema/include-non-file.err diff | blob | blame | history
tests/qapi-schema/include-non-file.json diff | blob | blame | history
Unnamed repository; edit this file 'description' to name the repository.