On Wed, 22 Jul 2026 17:36:43 GMT, Chris Plummer <[email protected]> wrote:

>> Rather saying "if preview is enabled", instead the text can just refer to 
>> the specific preview feature and then a link to a related preview API. This 
>> will result in a PREVIEW superscript being part of the link, clearly 
>> designating the feature as preview. For example:
>> 
>> 
>> -     * If preview features are enabled in the target VM, this method does 
>> not prevent a
>> -     * strictly-initialized field from being read before it has been 
>> initialized.
>> +     * This method does not prevent a
>> +     * {@linkplain java.lang.reflect.Field#isStrictInit() 
>> strictly-initialized field}
>> +     * from being read before it has been initialized.
>> 
>> 
>> Also, [JDK-8388113](https://bugs.openjdk.org/browse/JDK-8388113) was suppose 
>> to include some JDWP spec clarifying comments about strict field 
>> initialization behavior, but it was accidentally not included in the PR.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Chris Plummer has updated the pull request incrementally with two additional 
> commits since the last revision:
> 
>  - fix minor typo
>  - get rid of 'if preview mode is enabled'

My understanding is that `lworld` branch is read-only after we integrate 
JEP-401, so if this is for post integration it should be targeted to mainline.

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

PR Comment: https://git.openjdk.org/valhalla/pull/2658#issuecomment-5056036310

Reply via email to