On 04/30/2014 10:26 PM, Wenchao Xia wrote: > These cases will verify whether the expected qdict is built. > > Signed-off-by: Wenchao Xia <wenchaoq...@gmail.com> > --- > tests/Makefile | 14 ++- > tests/qapi-schema/qapi-schema-test.json | 12 ++ > tests/qapi-schema/qapi-schema-test.out | 10 +- > tests/test-qmp-event.c | 265 > +++++++++++++++++++++++++++++++ > 4 files changed, 296 insertions(+), 5 deletions(-) > create mode 100644 tests/test-qmp-event.c >
> @@ -232,12 +236,16 @@ $(SRC_PATH)/tests/qapi-schema/qapi-schema-test.json > $(SRC_PATH)/scripts/qapi-typ > tests/test-qapi-visit.c tests/test-qapi-visit.h :\ > $(SRC_PATH)/tests/qapi-schema/qapi-schema-test.json > $(SRC_PATH)/scripts/qapi-visit.py > $(call quiet-command,$(PYTHON) $(SRC_PATH)/scripts/qapi-visit.py > $(gen-out-type) -o tests -p "test-" < $<, " GEN $@") > +tests/test-qapi-event.c tests/test-qapi-event.h :\ > +$(SRC_PATH)/tests/qapi-schema/qapi-schema-test.json > $(SRC_PATH)/scripts/qapi-event.py > + $(call quiet-command,$(PYTHON) $(SRC_PATH)/scripts/qapi-event.py > $(gen-out-type) -o tests -p "test-" < $<, " GEN $@") I'd like to see LluĂs' modularization patches go in first, which means this should be rebased to break into shorter lines and to pass the name of the file as an input parameter rather than parsing it from stdin. https://lists.gnu.org/archive/html/qemu-devel/2014-04/msg05069.html But everything else here seems okay. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature