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