RFR: 8358015: Fix SequencedMap sequenced view method specifications [v2]
Stuart Marks
smarks at openjdk.org
Tue Jun 3 20:17:20 UTC 2025
On Mon, 2 Jun 2025 20:41:41 GMT, Joe Darcy <darcy at openjdk.org> wrote:
>> Stuart Marks has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Tweak nsee() declaration; adjust specs per suggestion; update copyrights.
>
> src/java.base/share/classes/java/util/SequencedMap.java line 296:
>
>> 294: return view().hashCode();
>> 295: }
>> 296: public void addFirst(K k) { throw new UnsupportedOperationException(); }
>
> Any utility in adding `@Override` annotations to the methods to document intent?
The `@Override` annotation has been inconsistently applied in the collections implementations. In practice since so many methods are overridden, and some test would likely fail if a method weren't overridden properly, the annotation would mostly add clutter. Indeed I'm considering pulling out all uses of `@Override` in certain areas because they're just clutter.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25515#discussion_r2124847105
More information about the core-libs-dev
mailing list