RFR: 8257776: [valhalla:jep390] Add disclaimer about future changes to value-based classes [v2]

Mandy Chung mchung at openjdk.java.net
Fri Dec 4 21:54:24 UTC 2020

On Fri, 4 Dec 2020 21:45:36 GMT, Dan Smith <dlsmith at openjdk.org> wrote:

>> Addressing CSR feedback by mentioning the possibility of future behavioral changes to value-based classes.
> Dan Smith has updated the pull request incrementally with one additional commit since the last revision:
>   Rephrase "future release" warning.

Marked as reviewed by mchung (Reviewer).

src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/MemoryAddress.java line 44:

> 42:  * All implementations of this interface must be <a href="{@docRoot}/java.base/java/lang/doc-files/ValueBased.html">value-based</a>;
> 43:  * programmers should treat instances that are {@linkplain #equals(Object) equal} as interchangeable and should not
> 44:  * use instances for synchronization, or unpredictable behavior may occur. For example, in a future release, synchronization may fail.

Nit: this line is very long and maybe good to wrap this line and the line above..   I understand that the javadoc in this file has long lines and href is an outliner.


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

More information about the valhalla-dev mailing list