Hi Paul,

On 2018-01-09 23:23, Paul Sandoz wrote:
This might also improve other tests using the same assert.

Not unlikely.


LambdaTestHelpers
--

s/if(/if (

+        int i = -1;
+        while (actual.hasNext() && expected.hasNext()) {
+            i++;
+            Object e = expected.next();
+            Object a = actual.next();
+            assertEquals(a, e, "Iterators differ");
+
+        }


‘i' is redundant. I presume might have wanted to use it to print out the index 
on element assertion failure, but it would of course induce garbage.

Fixed in place

/Claes

Reply via email to