RFR: 8257776: [valhalla:jep390] Add disclaimer about future changes to value-based classes [v2]
Dan Smith
dlsmith at openjdk.java.net
Fri Dec 4 22:27:24 UTC 2020
On Fri, 4 Dec 2020 21:49:07 GMT, Mandy Chung <mchung at openjdk.org> wrote:
>> Dan Smith has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Rephrase "future release" warning.
>
> 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.
Yeah, it's not conventional, but it's the style in this file, so I decided not to perturb it. This is an incubator module, I assume it will get polished up before becoming final.
-------------
PR: https://git.openjdk.java.net/valhalla/pull/294
More information about the valhalla-dev
mailing list