foreach: make test more accurate, fix for 64-bit gcc 4.4.3
authorRusty Russell <rusty@rustcorp.com.au>
Tue, 4 Oct 2011 01:39:45 +0000 (12:09 +1030)
committerRusty Russell <rusty@rustcorp.com.au>
Tue, 4 Oct 2011 01:39:45 +0000 (12:09 +1030)
gcc (Ubuntu 4.4.3-4ubuntu5) 4.4.3 on x86_64 uses a different stack
offset for the second call to test_ptr_recursion(), meaning it sees
new iterators.  Put it inside a loop to reduce the chance of that
happening.

ccan/foreach/test/run-nested.c

index b54fbbe07bcd1b17f658aa76845850d298657f3b..17fa2f16667c5cc881b00aa21f7c8dbf2b478e00 100644 (file)
@@ -47,7 +47,7 @@ int main(void)
        int i, j, sum;
        const char *istr, *jstr;
 
-       plan_tests(12);
+       plan_tests(16);
 
        sum = 0;
        foreach_int(i, 0, 1, 2, 3, 4)
@@ -68,12 +68,6 @@ int main(void)
        ok1(sum == 100);
        ok1(count_iters() <= 2);
 
-       sum = test_int_recursion(0);
-       diag("sum = %i\n", sum);
-       diag("iters = %i\n", count_iters());
-       ok1(sum == 160);
-       ok1(count_iters() <= 2 + 5); /* 5 is max depth of recursion. */
-
        sum = 0;
        foreach_ptr(istr, "0", "1", "2", "3", "4")
                foreach_ptr(jstr, "0", "1", "2", "3", "4")
@@ -81,7 +75,7 @@ int main(void)
        diag("sum = %i\n", sum);
        diag("iters = %i\n", count_iters());
        ok1(sum == 100);
-       ok1(count_iters() <= 2 + 5 + 2);
+       ok1(count_iters() <= 2 + 2);
 
        /* Same again... reusing iterators. */
        sum = 0;
@@ -91,13 +85,22 @@ int main(void)
        diag("sum = %i\n", sum);
        diag("iters = %i\n", count_iters());
        ok1(sum == 100);
-       ok1(count_iters() <= 2 + 5 + 2);
+       ok1(count_iters() <= 2 + 2);
 
-       sum = test_ptr_recursion("0");
-       diag("sum = %i\n", sum);
-       diag("iters = %i\n", count_iters());
-       ok1(sum == 160);
-       ok1(count_iters() <= 2 + 5 + 2);
+       /* Do this twice, second time shouldn't increase iterators. */
+       for (i = 0; i < 2; i++) {
+               sum = test_int_recursion(0);
+               diag("sum = %i\n", sum);
+               diag("iters = %i\n", count_iters());
+               ok1(sum == 160);
+               ok1(count_iters() <= 2 + 2 + 5); /* 5 is max depth of recursion. */
+
+               sum = test_ptr_recursion("0");
+               diag("sum = %i\n", sum);
+               diag("iters = %i\n", count_iters());
+               ok1(sum == 160);
+               ok1(count_iters() <= 2 + 2 + 5);
+       }
        return exit_status();
 }