On Tue, 28 Apr 2026 11:20:27 GMT, Daniel Fuchs <[email protected]> wrote:
>> sstremler has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Delete unnecessary whitespace >> >> Co-authored-by: Andrey Turbanov <[email protected]> >> - Delete unnecessary whitespace >> >> Co-authored-by: Andrey Turbanov <[email protected]> > > test/jdk/java/util/BitSet/stream/BitSetStreamTest.java line 124: > >> 122: for (int i = bs.nextSetBit(0); i >= 0; i = bs.nextSetBit(i + >> 1)) { >> 123: assertTrue(it.hasNext()); >> 124: assertEquals(it.nextInt(), i); > > I believe the arguments need to be swapped here: it should be > `assertEquals(expected, actual)` Could you double check whether other places where assertEquals is called need their arguments to be swapped? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30954#discussion_r3153665548
