[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] libqtest: New qtest_end() to go with qtest_
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] libqtest: New qtest_end() to go with qtest_start() |
Date: |
Wed, 19 Jun 2013 14:36:47 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6 |
Am 19.06.2013 09:44, schrieb Markus Armbruster:
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
> tests/fdc-test.c | 2 +-
> tests/hd-geo-test.c | 8 ++++----
> tests/ide-test.c | 2 +-
> tests/libqtest.h | 7 +++++++
> 4 files changed, 13 insertions(+), 6 deletions(-)
>
> diff --git a/tests/fdc-test.c b/tests/fdc-test.c
> index 4b0301d..fd198dc 100644
> --- a/tests/fdc-test.c
> +++ b/tests/fdc-test.c
> @@ -556,7 +556,7 @@ int main(int argc, char **argv)
> ret = g_test_run();
>
> /* Cleanup */
> - qtest_quit(global_qtest);
> + qtest_end();
> unlink(test_image);
>
> return ret;
> diff --git a/tests/hd-geo-test.c b/tests/hd-geo-test.c
> index 9a31e85..b72042e 100644
> --- a/tests/hd-geo-test.c
> +++ b/tests/hd-geo-test.c
> @@ -244,7 +244,7 @@ static void test_ide_none(void)
> setup_common(argv, ARRAY_SIZE(argv));
> qtest_start(g_strjoinv(" ", argv));
> test_cmos();
> - qtest_quit(global_qtest);
> + qtest_end();
> }
>
> static void test_ide_mbr(bool use_device, MBRcontents mbr)
> @@ -262,7 +262,7 @@ static void test_ide_mbr(bool use_device, MBRcontents mbr)
> }
> qtest_start(g_strjoinv(" ", argv));
> test_cmos();
> - qtest_quit(global_qtest);
> + qtest_end();
> }
>
> /*
> @@ -334,7 +334,7 @@ static void test_ide_drive_user(const char *dev, bool
> trans)
> g_free(opts);
> qtest_start(g_strjoinv(" ", argv));
> test_cmos();
> - qtest_quit(global_qtest);
> + qtest_end();
> }
>
> /*
> @@ -387,7 +387,7 @@ static void test_ide_drive_cd_0(void)
> }
> qtest_start(g_strjoinv(" ", argv));
> test_cmos();
> - qtest_quit(global_qtest);
> + qtest_end();
> }
>
> int main(int argc, char **argv)
> diff --git a/tests/ide-test.c b/tests/ide-test.c
> index 7e2eb94..7307f1d 100644
> --- a/tests/ide-test.c
> +++ b/tests/ide-test.c
> @@ -122,7 +122,7 @@ static void ide_test_start(const char *cmdline_fmt, ...)
>
> static void ide_test_quit(void)
> {
> - qtest_quit(global_qtest);
> + qtest_end();
> }
>
> static QPCIDevice *get_pci_device(uint16_t *bmdma_base)
> diff --git a/tests/libqtest.h b/tests/libqtest.h
> index 437bda3..329325b 100644
> --- a/tests/libqtest.h
> +++ b/tests/libqtest.h
> @@ -17,6 +17,7 @@
> #ifndef LIBQTEST_H
> #define LIBQTEST_H
>
> +#include <stddef.h>
> #include <stdint.h>
> #include <stdbool.h>
> #include <stdarg.h>
> @@ -318,6 +319,12 @@ static inline QTestState *qtest_start(const char *args)
> return global_qtest;
> }
>
> +static inline void qtest_end(void)
> +{
> + qtest_quit(global_qtest);
> + global_qtest = NULL;
> +}
> +
> /**
> * qmp:
> * @fmt...: QMP message to send to qemu
>
Function and usage looks fine, but please add a gtk-doc comment.
qtest_start() does have one.
Thanks,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg