tests: add test_seat_release() for symmetry
Add test_seat_release() as the counterpart of test_seat_init() instead
of open-coding it. This helps adding more code to test_seat_release()
later.
Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
diff --git a/tests/weston-test.c b/tests/weston-test.c
index 73409ca..ae08b02 100644
--- a/tests/weston-test.c
+++ b/tests/weston-test.c
@@ -80,6 +80,9 @@
static int
test_seat_init(struct weston_test *test)
{
+ assert(!test->is_seat_initialized &&
+ "Trying to add already added test seat");
+
/* create our own seat */
weston_seat_init(&test->seat, test->compositor, "test-seat");
test->is_seat_initialized = true;
@@ -93,6 +96,16 @@
return 0;
}
+static void
+test_seat_release(struct weston_test *test)
+{
+ assert(test->is_seat_initialized &&
+ "Trying to release already released test seat");
+ test->is_seat_initialized = false;
+ weston_seat_release(&test->seat);
+ memset(&test->seat, 0, sizeof test->seat);
+}
+
static struct weston_seat *
get_seat(struct weston_test *test)
{
@@ -270,10 +283,7 @@
} else if (strcmp(device, "touch") == 0) {
weston_seat_release_touch(seat);
} else if (strcmp(device, "seat") == 0) {
- assert(test->is_seat_initialized &&
- "Trying to release already released test seat");
- weston_seat_release(seat);
- test->is_seat_initialized = false;
+ test_seat_release(test);
} else {
assert(0 && "Unsupported device");
}
@@ -293,8 +303,6 @@
} else if (strcmp(device, "touch") == 0) {
weston_seat_init_touch(seat);
} else if (strcmp(device, "seat") == 0) {
- assert(!test->is_seat_initialized &&
- "Trying to add already added test seat");
test_seat_init(test);
} else {
assert(0 && "Unsupported device");
