[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH for-next v3 4/5] libqtest: Introduce qtest_qmpv() an
From: |
Andreas Färber |
Subject: |
[Qemu-devel] [PATCH for-next v3 4/5] libqtest: Introduce qtest_qmpv() and convert remaining macro |
Date: |
Mon, 11 Feb 2013 20:01:21 +0100 |
In order to convert qmp() macro to an inline function, expose a
qtest_qmpv() function, reused by qtest_qmp().
We can't apply GCC_FMT_ATTR() since fdc-test is using zero-length format
strings, which would result in warnings treated as errors.
Signed-off-by: Andreas Färber <address@hidden>
---
tests/libqtest.c | 14 ++++++++++----
tests/libqtest.h | 20 +++++++++++++++++++-
2 Dateien geändert, 29 Zeilen hinzugefügt(+), 5 Zeilen entfernt(-)
diff --git a/tests/libqtest.c b/tests/libqtest.c
index 762dec4..da58ff5 100644
--- a/tests/libqtest.c
+++ b/tests/libqtest.c
@@ -288,16 +288,13 @@ redo:
return words;
}
-void qtest_qmp(QTestState *s, const char *fmt, ...)
+void qtest_qmpv(QTestState *s, const char *fmt, va_list ap)
{
- va_list ap;
bool has_reply = false;
int nesting = 0;
/* Send QMP request */
- va_start(ap, fmt);
socket_sendf(s->qmp_fd, fmt, ap);
- va_end(ap);
/* Receive reply */
while (!has_reply || nesting > 0) {
@@ -326,6 +323,15 @@ void qtest_qmp(QTestState *s, const char *fmt, ...)
}
}
+void qtest_qmp(QTestState *s, const char *fmt, ...)
+{
+ va_list ap;
+
+ va_start(ap, fmt);
+ qtest_qmpv(s, fmt, ap);
+ va_end(ap);
+}
+
const char *qtest_get_arch(void)
{
const char *qemu = getenv("QTEST_QEMU_BINARY");
diff --git a/tests/libqtest.h b/tests/libqtest.h
index a111c9c..f5c6e21 100644
--- a/tests/libqtest.h
+++ b/tests/libqtest.h
@@ -17,6 +17,7 @@
#include <stdint.h>
#include <stdbool.h>
+#include <stdarg.h>
#include <sys/types.h>
typedef struct QTestState QTestState;
@@ -49,6 +50,16 @@ void qtest_quit(QTestState *s);
void qtest_qmp(QTestState *s, const char *fmt, ...);
/**
+ * qtest_qmpv:
+ * @s: #QTestState instance to operate on.
+ * @fmt: QMP message to send to QEMU
+ * @ap: QMP message arguments
+ *
+ * Sends a QMP message to QEMU.
+ */
+void qtest_qmpv(QTestState *s, const char *fmt, va_list ap);
+
+/**
* qtest_get_irq:
* @s: #QTestState instance to operate on.
* @num: Interrupt to observe.
@@ -227,7 +238,14 @@ static inline QTestState *qtest_start(const char *args)
*
* Sends a QMP message to QEMU
*/
-#define qmp(fmt, ...) qtest_qmp(global_qtest, fmt, ## __VA_ARGS__)
+static inline void qmp(const char *fmt, ...)
+{
+ va_list ap;
+
+ va_start(ap, fmt);
+ qtest_qmpv(global_qtest, fmt, ap);
+ va_end(ap);
+}
/**
* get_irq:
--
1.7.10.4
- [Qemu-devel] [PATCH for-next v3 0/5] qtest: tmp105 cleanups and MMIO support, Andreas Färber, 2013/02/11
- [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Andreas Färber, 2013/02/11
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Andreas Färber, 2013/02/14
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Stefan Hajnoczi, 2013/02/15
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Andreas Färber, 2013/02/15
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Stefan Hajnoczi, 2013/02/15
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Andreas Färber, 2013/02/15
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Alexander Graf, 2013/02/15
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Andreas Färber, 2013/02/16
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Paolo Bonzini, 2013/02/17
[Qemu-devel] [PATCH for-next v3 4/5] libqtest: Introduce qtest_qmpv() and convert remaining macro,
Andreas Färber <=
[Qemu-devel] [PATCH for-next v3 3/5] libqtest: Convert macros to functions and clean up documentation, Andreas Färber, 2013/02/11
[Qemu-devel] [PATCH for-next v3 1/5] tmp105-test: Combine assertions of 16-bit responses, Andreas Färber, 2013/02/11
[Qemu-devel] [PATCH for-next v3 5/5] qtest: Add MMIO support, Andreas Färber, 2013/02/11