RFR: 8338411: Implement JEP 486: Permanently Disable the Security Manager [v3]

Alexey Ivanov aivanov at openjdk.org
Fri Nov 1 19:17:12 UTC 2024


On Mon, 28 Oct 2024 14:08:46 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> src/java.desktop/share/classes/java/awt/Font.java line 1612:
>> 
>>> 1610:      * obtained.  The {@code String} value of this property is then
>>> 1611:      * interpreted as a {@code Font} object according to the
>>> 1612:      * specification of {@code Font.decode(String)}
>> 
>> Suggestion:
>> 
>>      * specification of {@code Font.decode(String)}.
>> 
>> Period is missing.
>
> Not part of this change, can be fixed later as a follow-on cleanup.

I've submitted [JDK-8343446](https://bugs.openjdk.org/browse/JDK-8343446): _Add missing periods in docs for java.awt.Font_.

>> src/java.desktop/share/classes/java/awt/Font.java line 1780:
>> 
>>> 1778:      * <p>
>>> 1779:      * The property value should be one of the forms accepted by
>>> 1780:      * {@code Font.decode(String)}
>> 
>> Suggestion:
>> 
>>      * {@code Font.decode(String)}.
>> 
>> Period is missing.
>
> Not part of this change, can be fixed later as a follow-on cleanup.

Deferred to [JDK-8343446](https://bugs.openjdk.org/browse/JDK-8343446).

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21498#discussion_r1826080609
PR Review Comment: https://git.openjdk.org/jdk/pull/21498#discussion_r1826081342


More information about the compiler-dev mailing list