RFR: 8199318: add idempotent copy operation for Map.Entry [v2]

Daniel Fuchs dfuchs at openjdk.java.net
Wed Jun 2 18:10:35 UTC 2021


On Wed, 2 Jun 2021 17:54:06 GMT, Stuart Marks <smarks at openjdk.org> wrote:

>> I'm fixing this along with a couple intertwined issues.
>> 
>> 1. Add Map.Entry::copyOf as an idempotent copy operation.
>> 
>> 2. Clarify that AbstractMap.SimpleImmutableEntry is itself unmodifiable (not really immutable) but that subclasses can be modifiable.
>> 
>> 3. Clarify some confusing, historical wording about Map.Entry instances being obtainable only via iteration of a Map's entry-set view. This was never really true, since anyone could implement the Map.Entry interface, and it certainly was not true since JDK 1.6 when SimpleEntry and SimpleImmutableEntry were added.
>
> Stuart Marks has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Tiny editorial tweaks.

src/java.base/share/classes/java/util/Map.java line 396:

> 394: 
> 395:     /**
> 396:      * A map entry (key-value pair). The Entry may be unmodifiable, or the

In that case then maybe it should be `{@code Entry}` in both places where `entry` was replaced with `Entry` ?

-------------

PR: https://git.openjdk.java.net/jdk/pull/4295


More information about the core-libs-dev mailing list