On Fri, 16 May 2025 09:24:13 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
>> Stuart Marks has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Reversed view of SynchronizedRandomAccessList should also be RandomAccess. >> Add tests to ensure RandomAccess is preserved when reversing. > > test/jdk/java/util/Collections/SyncListBash.java line 31: > >> 29: * @run main SyncListBash f >> 30: * @run main SyncListBash r >> 31: * @summary Test that List's new SequencedCollection methods are properly > > Nit - jtreg recommends `@summary` to be after `@bug` > https://openjdk.org/jtreg/tag-spec.html#ORDER. It would be good to follow > that for this new test. Oh, thanks for pointing this out. I'll fix. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/24990#discussion_r2093806293
