RFR: JDK-8315876 Open source several Swing CSS related tests [v4]
Alexey Ivanov
aivanov at openjdk.org
Tue Sep 19 17:40:45 UTC 2023
On Tue, 19 Sep 2023 16:55:02 GMT, Harshitha Onkar <honkar at openjdk.org> wrote:
>> test/jdk/javax/swing/text/html/HTMLDocument/bug4209280.java line 36:
>>
>>> 34: public static void main(String[] args) throws Exception {
>>> 35: String html = "<html><bold>Foo</bold></html>";
>>> 36: new JLabel(html);
>>
>> Suggestion:
>>
>> // The following line should throw no exceptions
>> new JLabel(html);
>>
>> This may clarify the intention of the reader skipped the summary.
>
> Agreed. Adds more clarity. Updated.
Usually full stops aren't used in comments, unless it consists of several sentences. I'm nitpicking.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15769#discussion_r1330489466
More information about the client-libs-dev
mailing list