Integrated: 8199318: add idempotent copy operation for Map.Entry

Stuart Marks smarks at openjdk.java.net
Fri Jun 4 17:16:03 UTC 2021


On Wed, 2 Jun 2021 00:39:25 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.

This pull request has now been integrated.

Changeset: cd0678fc
Author:    Stuart Marks <smarks at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/cd0678fcf6bc00ecda3e61d959617c67d02dba3c
Stats:     141 lines in 3 files changed: 120 ins; 2 del; 19 mod

8199318: add idempotent copy operation for Map.Entry

Reviewed-by: alanb, psandoz, dfuchs

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

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


More information about the core-libs-dev mailing list