[PATCH wayland 4/4] array-test: Include wayland-util.h and simplify init test

Yong Bakos junk at humanoriented.com
Fri Sep 16 22:37:39 UTC 2016


From: Yong Bakos <ybakos at humanoriented.com>

Include wayland-util.h instead of wayland-private.h, because that's where
wl_array is and nothing in wayland-private.h is required.

Remove the useless repeated testing of wl_array_init, because if it fails once
out of thousands of iterations we're all doomed anyway.

Signed-off-by: Yong Bakos <ybakos at humanoriented.com>
---
 tests/array-test.c | 20 ++++++--------------
 1 file changed, 6 insertions(+), 14 deletions(-)

diff --git a/tests/array-test.c b/tests/array-test.c
index a5856fd..4aaf041 100644
--- a/tests/array-test.c
+++ b/tests/array-test.c
@@ -25,24 +25,16 @@
 
 #include <stdlib.h>
 #include <assert.h>
-#include "wayland-private.h"
+#include "wayland-util.h"
 #include "test-runner.h"
 
 TEST(array_init)
 {
-	const int iterations = 4122; /* this is arbitrary */
-	int i;
-
-	/* Init array an arbitray amount of times and verify the
-	 * defaults are sensible. */
-
-	for (i = 0; i < iterations; i++) {
-		struct wl_array array;
-		wl_array_init(&array);
-		assert(array.size == 0);
-		assert(array.alloc == 0);
-		assert(array.data == 0);
-	}
+	struct wl_array array;
+	wl_array_init(&array);
+	assert(array.size == 0);
+	assert(array.alloc == 0);
+	assert(array.data == 0);
 }
 
 TEST(array_release)
-- 
2.7.2



More information about the wayland-devel mailing list