RFR: 8254275: Development to revise "value-based class" & apply to wrappers [v2]

Dan Smith dlsmith at openjdk.java.net
Tue Oct 13 22:59:36 UTC 2020


> 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

Dan Smith has updated the pull request incrementally with one additional commit since the last revision:

  Revise definition for more flexible ==. Apply revised boilerplate to wrappers and existing references.

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

Changes:
  - all: https://git.openjdk.java.net/valhalla/pull/222/files
  - new: https://git.openjdk.java.net/valhalla/pull/222/files/5ae32ad8..0b97d4ab

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=valhalla&pr=222&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=valhalla&pr=222&range=00-01

  Stats: 265 lines in 46 files changed: 70 ins; 32 del; 163 mod
  Patch: https://git.openjdk.java.net/valhalla/pull/222.diff
  Fetch: git fetch https://git.openjdk.java.net/valhalla pull/222/head:pull/222

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



More information about the valhalla-dev mailing list