Re: [Qemu-devel] [PATCH v2 2/3] qtest: extend qtest_qmp() to fill in the reply

2013-01-07 Thread Anthony Liguori
Jason Baron jba...@redhat.com writes: From: Jason Baron jba...@redhat.com Introduce: Add void qtest_qmp_resp(QTestState *s, QString *resp, const char *fmt, ...) Any reason to not just return a QString? Also, why not do the parsing in this function and just return a QObject of the result?

[Qemu-devel] [PATCH v2 2/3] qtest: extend qtest_qmp() to fill in the reply

2012-12-20 Thread Jason Baron
From: Jason Baron jba...@redhat.com Introduce: Add void qtest_qmp_resp(QTestState *s, QString *resp, const char *fmt, ...) which allows a response string to be filled in. Signed-off-by: Jason Baron jba...@redhat.com --- tests/Makefile |2 +- tests/libqtest.c | 17 ++---