From: Chris Wilson <chris@chris-wilson.co.uk>
Date: Mon, 18 Nov 2013 12:04:29 +0000 (+0000)
Subject: tests: Avoid dependence on undefined C
X-Git-Tag: 2.99.907~88
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e99e643ed1f2856109426f791a7615ee122d7c42;p=platform%2Fupstream%2Fxf86-video-intel.git

tests: Avoid dependence on undefined C

Right-shifting by a negative value is undefined.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---

diff --git a/test/basic-copyarea-size.c b/test/basic-copyarea-size.c
index 732bb81..5e3373d 100644
--- a/test/basic-copyarea-size.c
+++ b/test/basic-copyarea-size.c
@@ -57,7 +57,7 @@ int main(int argc, char **argv)
 
 		printf("size=%d\n", size);
 		for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-			int reps = 1 << i;
+			int reps = REPS(i);
 			do {
 				int sx = rand() % (2*size) - size;
 				int sy = rand() % (2*size) - size;
diff --git a/test/basic-copyarea.c b/test/basic-copyarea.c
index 025b884..8b64199 100644
--- a/test/basic-copyarea.c
+++ b/test/basic-copyarea.c
@@ -282,13 +282,10 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
+		int reps = REPS(i);
+		int sets = SETS(i);
 		enum target t;
 
-		if (sets < 2)
-			sets = 2;
-
 		for (t = TARGET_FIRST; t <= TARGET_LAST; t++) {
 			pixel_tests(&test, reps, sets, t);
 			area_tests(&test, reps, sets, t);
diff --git a/test/basic-fillrect.c b/test/basic-fillrect.c
index 67d7067..186604c 100644
--- a/test/basic-fillrect.c
+++ b/test/basic-fillrect.c
@@ -246,13 +246,10 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
+		int reps = REPS(i);
+		int sets = SETS(i);
 		enum target t;
 
-		if (sets < 2)
-			sets = 2;
-
 		for (t = TARGET_FIRST; t <= TARGET_LAST; t++) {
 			pixel_tests(&test, reps, sets, t);
 			area_tests(&test, reps, sets, t);
diff --git a/test/basic-putimage.c b/test/basic-putimage.c
index 5d68908..10053bc 100644
--- a/test/basic-putimage.c
+++ b/test/basic-putimage.c
@@ -262,13 +262,10 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
+		int reps = REPS(i);
+		int sets = SETS(i);
 		enum target t;
 
-		if (sets < 2)
-			sets = 2;
-
 		for (t = TARGET_FIRST; t <= TARGET_LAST; t++) {
 			pixel_tests(&test, reps, sets, t, 0);
 			area_tests(&test, reps, sets, t, 0);
diff --git a/test/basic-rectangle.c b/test/basic-rectangle.c
index 8f78bba..8de200c 100644
--- a/test/basic-rectangle.c
+++ b/test/basic-rectangle.c
@@ -204,13 +204,9 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
+		int reps = REPS(i), sets = SETS(i);
 		enum target t;
 
-		if (sets < 2)
-			sets = 2;
-
 		for (t = TARGET_FIRST; t <= TARGET_LAST; t++) {
 			zrect_tests(&test, reps, sets, t);
 			hrect_tests(&test, reps, sets, t);
diff --git a/test/basic-stress.c b/test/basic-stress.c
index 5657173..4b42865 100644
--- a/test/basic-stress.c
+++ b/test/basic-stress.c
@@ -147,7 +147,7 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int iterations = 1 << i;
+		int iterations = REPS(i);
 		rect_tests(&test, iterations, 0);
 		rect_tests(&test, iterations, 1);
 	}
diff --git a/test/basic-string.c b/test/basic-string.c
index 9f59c91..1e24b29 100644
--- a/test/basic-string.c
+++ b/test/basic-string.c
@@ -86,13 +86,9 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
+		int reps = REPS(i), sets = SETS(i);
 		enum target t;
 
-		if (sets < 2)
-			sets = 2;
-
 		for (t = TARGET_FIRST; t <= TARGET_LAST; t++) {
 			string_tests(&test, reps, sets, t);
 		}
diff --git a/test/mixed-stress.c b/test/mixed-stress.c
index 42e2c08..a315a9c 100644
--- a/test/mixed-stress.c
+++ b/test/mixed-stress.c
@@ -200,7 +200,7 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int iterations = 1 << i;
+		int iterations = REPS(i);
 		rect_tests(&test, iterations, 0);
 		rect_tests(&test, iterations, 1);
 	}
diff --git a/test/render-composite-solid-mask.c b/test/render-composite-solid-mask.c
index b86512c..925df3c 100644
--- a/test/render-composite-solid-mask.c
+++ b/test/render-composite-solid-mask.c
@@ -107,13 +107,9 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
+		int reps = REPS(i), sets = SETS(i);
 		enum target t;
 
-		if (sets < 2)
-			sets = 2;
-
 		for (t = TARGET_FIRST; t <= TARGET_LAST; t++)
 			ref_tests(&test, reps, sets, t);
 	}
diff --git a/test/render-composite-solid.c b/test/render-composite-solid.c
index 6609c4b..b4742af 100644
--- a/test/render-composite-solid.c
+++ b/test/render-composite-solid.c
@@ -238,13 +238,9 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
+		int reps = REPS(i), sets = SETS(i);
 		enum target t;
 
-		if (sets < 2)
-			sets = 2;
-
 		for (t = TARGET_FIRST; t <= TARGET_LAST; t++) {
 			pixel_tests(&test, reps, sets, t);
 			area_tests(&test, reps, sets, t);
diff --git a/test/render-copy-alphaless.c b/test/render-copy-alphaless.c
index b968704..5f1487e 100644
--- a/test/render-copy-alphaless.c
+++ b/test/render-copy-alphaless.c
@@ -270,11 +270,7 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
-
-		if (sets < 2)
-			sets = 2;
+		int reps = REPS(i), sets = SETS(i);
 
 		pixel_tests(&test, reps, sets, PIXMAP);
 		area_tests(&test, reps, sets, PIXMAP);
diff --git a/test/render-copyarea-mask.c b/test/render-copyarea-mask.c
index c8bb715..5116c16 100644
--- a/test/render-copyarea-mask.c
+++ b/test/render-copyarea-mask.c
@@ -149,13 +149,9 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
+		int reps = REPS(i), sets = SETS(i);
 		enum target t;
 
-		if (sets < 2)
-			sets = 2;
-
 		for (t = TARGET_FIRST; t <= TARGET_LAST; t++) {
 			rect_tests(&test, reps, sets, t, 0);
 			if (t != PIXMAP)
diff --git a/test/render-copyarea-size.c b/test/render-copyarea-size.c
index 65fb6ca..55e3b56 100644
--- a/test/render-copyarea-size.c
+++ b/test/render-copyarea-size.c
@@ -65,7 +65,7 @@ int main(int argc, char **argv)
 
 		printf("size=%d\n", size);
 		for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-			int reps = 1 << i;
+			int reps = REPS(i);
 			do {
 				int sx = rand() % (2*size) - size;
 				int sy = rand() % (2*size) - size;
diff --git a/test/render-copyarea.c b/test/render-copyarea.c
index bcab553..8c00b34 100644
--- a/test/render-copyarea.c
+++ b/test/render-copyarea.c
@@ -305,13 +305,9 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
+		int reps = REPS(i), sets = SETS(i);
 		enum target t;
 
-		if (sets < 2)
-			sets = 2;
-
 		for (t = TARGET_FIRST; t <= TARGET_LAST; t++) {
 			pixel_tests(&test, reps, sets, t);
 			area_tests(&test, reps, sets, t);
diff --git a/test/render-fill-copy.c b/test/render-fill-copy.c
index bc4a734..45551b2 100644
--- a/test/render-fill-copy.c
+++ b/test/render-fill-copy.c
@@ -262,13 +262,9 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
+		int reps = REPS(i), sets = SETS(i);
 		enum target t;
 
-		if (sets < 2)
-			sets = 2;
-
 		for (t = TARGET_FIRST; t <= TARGET_LAST; t++) {
 			pixel_tests(&test, reps, sets, t);
 			area_tests(&test, reps, sets, t);
diff --git a/test/render-fill.c b/test/render-fill.c
index 27d565b..54be1a3 100644
--- a/test/render-fill.c
+++ b/test/render-fill.c
@@ -230,13 +230,9 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
+		int reps = REPS(i), sets = SETS(i);
 		enum target t;
 
-		if (sets < 2)
-			sets = 2;
-
 		for (t = TARGET_FIRST; t <= TARGET_LAST; t++) {
 			pixel_tests(&test, reps, sets, t);
 			area_tests(&test, reps, sets, t);
diff --git a/test/render-trapezoid-image.c b/test/render-trapezoid-image.c
index c794cbb..c25661d 100644
--- a/test/render-trapezoid-image.c
+++ b/test/render-trapezoid-image.c
@@ -585,11 +585,7 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
-
-		if (sets < 2)
-			sets = 2;
+		int reps = REPS(i), SETS(i);
 
 		for (target = TARGET_FIRST; target <= TARGET_LAST; target++) {
 			pixel_tests(&test, reps, sets, target, 0);
diff --git a/test/render-trapezoid.c b/test/render-trapezoid.c
index f7fd5bb..811d909 100644
--- a/test/render-trapezoid.c
+++ b/test/render-trapezoid.c
@@ -414,11 +414,7 @@ int main(int argc, char **argv)
 	test_init(&test, argc, argv);
 
 	for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
-		int reps = 1 << i;
-		int sets = 1 << (12 - i);
-
-		if (sets < 2)
-			sets = 2;
+		int reps = REPS(i), sets = SETS(i);
 
 		for (target = TARGET_FIRST; target <= TARGET_LAST; target++) {
 			pixel_tests(&test, reps, sets, target);
diff --git a/test/test.h b/test/test.h
index 7ef4dca..3cd9457 100644
--- a/test/test.h
+++ b/test/test.h
@@ -120,4 +120,7 @@ double test_timer_stop(struct test_display *t, struct timespec *tv);
 #define ARRAY_SIZE(a) (sizeof(a) / sizeof(a[0]))
 #endif
 
+#define REPS(I) (1 << (I))
+#define SETS(I) ((I) < 12 ? 1 << (12 - (I)) : 2)
+
 #endif