On Thu, 23 Jul 2026 20:55:30 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 one additional 
> commit since the last revision:
> 
>   add 'preview mode' comments back in

You've got this to a good place.

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

Marked as reviewed by alanb (no project role).

PR Review: 
https://git.openjdk.org/valhalla/pull/2658#pullrequestreview-4773193399

Reply via email to