On Thu, 1 Jun 2023 16:18:12 GMT, Stuart Marks <sma...@openjdk.org> wrote:
> Adjust the specification of the `SequencedMap` sequenced-view methods, and
> adjust implementations to match.
src/java.base/share/classes/java/util/AbstractMap.java line 908:
> 906: public boolean add(E t) { throw uoe(); }
> 907: public boolean addAll(Collection<? extends E> c) { throw uoe(); }
> 908: public void clear() { view().clear(); }
Each of these methods should perhaps have the `@Override` to make it clear that
these correspond to the interface this class is implementing. But I see that
the `AbstractMap` class itself doesn't use `@Override` anywhere, so I think
it's OK to keep it consistent in its current form in this PR.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14267#discussion_r1218900019
