RFR: 8134459: java/util/stream/test/org/openjdk/tests/java/util/stream/WhileOpTest.java timed out
Claes Redestad
claes.redestad at oracle.com
Tue Jan 9 22:55:37 UTC 2018
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
More information about the core-libs-dev
mailing list