On Fri, 30 May 2025 14:33:05 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
>> For a full explanation, see the bug report >> [JDK-8358015](https://bugs.openjdk.org/browse/JDK-8358015). >> >> This PR includes three related changes: >> * New overrides in SequencedMap view collection implementations, which >> improve their behavior. >> * Update to `@implSpec` clauses to reflect this behavior (which was >> previously ill-specified). >> * Update to a test which relied on unspecified behavior. > > src/java.base/share/classes/java/util/SequencedMap.java line 271: > >> 269: * Its {@link SequencedSet#getFirst getFirst} and {@link >> SequencedSet#getLast getLast} >> 270: * methods are implemented in terms of the {@link #firstEntry >> firstEntry} and {@link >> 271: * #lastEntry lastEntry} methods of this class, respectively. Its >> {@link > > Hello Stuart, some parts of this and other updated method docs use "this > interface", "this map", "this class". Was it intentional to use "this class" > in this newly added text? Good point. The other `@implSpec` clauses here all say "this interface" so I'll change the newly added "this class" to "this interface" in order to try to avoid confusion. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25515#discussion_r2116834181