RFR: 8254275: Development to revise ValueBased.html for consistency with inline class migration

[Mandy Chung]

On Fri, 9 Oct 2020 20:26:31 GMT, Dan Smith <dlsmith at openjdk.org> wrote:

> Polishing the specification of "value-based class" to align with requirements of inline classes, allow classes (like
> Integer) with deprecated constructors, and clarify expectations for clients.
> Here are the files that link to ValueBased.html:
> 
> % -> grep -lR "ValueBased.html" src
> src/java.base/share/classes/java/util/Map.java
> src/java.base/share/classes/java/util/Optional.java
> src/java.base/share/classes/java/util/OptionalDouble.java
> src/java.base/share/classes/java/util/KeyValueHolder.java
> src/java.base/share/classes/java/util/List.java
> src/java.base/share/classes/java/util/OptionalInt.java
> src/java.base/share/classes/java/util/OptionalLong.java
> src/java.base/share/classes/java/util/Set.java
> src/java.base/share/classes/java/time/OffsetTime.java
> src/java.base/share/classes/java/time/ZonedDateTime.java
> src/java.base/share/classes/java/time/Duration.java
> src/java.base/share/classes/java/time/Instant.java
> src/java.base/share/classes/java/time/chrono/MinguoDate.java
> src/java.base/share/classes/java/time/chrono/HijrahDate.java
> src/java.base/share/classes/java/time/chrono/JapaneseDate.java
> src/java.base/share/classes/java/time/chrono/ThaiBuddhistDate.java
> src/java.base/share/classes/java/time/LocalDateTime.java
> src/java.base/share/classes/java/time/LocalTime.java
> src/java.base/share/classes/java/time/YearMonth.java
> src/java.base/share/classes/java/time/Year.java
> src/java.base/share/classes/java/time/MonthDay.java
> src/java.base/share/classes/java/time/OffsetDateTime.java
> src/java.base/share/classes/java/time/ZoneId.java
> src/java.base/share/classes/java/time/LocalDate.java
> src/java.base/share/classes/java/time/Period.java
> src/java.base/share/classes/java/time/ZoneOffset.java
> src/java.base/share/classes/java/lang/ProcessHandle.java
> src/java.base/share/classes/java/lang/constant/DynamicCallSiteDesc.java
> src/java.base/share/classes/java/lang/constant/DynamicConstantDesc.java
> src/java.base/share/classes/java/lang/constant/ConstantDesc.java
> src/java.base/share/classes/java/lang/Runtime.java
> src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/ValueLayout.java
> src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/MappedMemorySegment.java
> src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/GroupLayout.java
> src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/MemoryLayout.java
> src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/MemoryAddress.java
> src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/PaddingLayout.java
> src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/MemorySegment.java
> src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/SequenceLayout.java

This looks fine to me.

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

Marked as reviewed by mchung (Reviewer).

PR: https://git.openjdk.java.net/valhalla/pull/222