diff options
author | Markus Armbruster <armbru@redhat.com> | 2018-08-06 08:53:43 +0200 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2018-08-16 08:42:06 +0200 |
commit | 88b988c895e3c226e264502cec03e13c34bb8f61 (patch) | |
tree | 4aa07775e66b6c7b69f9dde4c35d77d1871b0634 /tests/libqtest.c | |
parent | e3dc93be1ac46fdb58142383319b783b4c4ce8ca (diff) |
libqtest: Replace qtest_startf() by qtest_initf()
qtest_init() creates a new QTestState, and leaves @global_qtest alone.
qtest_start() additionally assigns it to @global_qtest, but
qtest_startf() additionally assigns NULL to @global_qtest. This makes
no sense. Replace it by qtest_initf() that works like qtest_init(),
i.e. leaves @global_qtest alone.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20180806065344.7103-23-armbru@redhat.com>
Diffstat (limited to 'tests/libqtest.c')
-rw-r--r-- | tests/libqtest.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/tests/libqtest.c b/tests/libqtest.c index 2f81bc6382..a0d44793fa 100644 --- a/tests/libqtest.c +++ b/tests/libqtest.c @@ -259,24 +259,23 @@ QTestState *qtest_init(const char *extra_args) return s; } -QTestState *qtest_vstartf(const char *fmt, va_list ap) +QTestState *qtest_vinitf(const char *fmt, va_list ap) { char *args = g_strdup_vprintf(fmt, ap); QTestState *s; - s = qtest_start(args); + s = qtest_init(args); g_free(args); - global_qtest = NULL; return s; } -QTestState *qtest_startf(const char *fmt, ...) +QTestState *qtest_initf(const char *fmt, ...) { va_list ap; QTestState *s; va_start(ap, fmt); - s = qtest_vstartf(fmt, ap); + s = qtest_vinitf(fmt, ap); va_end(ap); return s; } |