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