RFR: 8340466: Add description for PassFailJFrame constructors [v2]

Alexey Ivanov aivanov at openjdk.org
Wed Sep 25 10:39:09 UTC 2024


On Tue, 24 Sep 2024 17:20:57 GMT, Phil Race <prr at openjdk.org> wrote:

>> Alexey Ivanov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use EDT abbreviation in constructor description
>>   
>>   The term EDT (Event Dispatch Thread) is introduced in
>>   the top-level doc comment.
>
> test/jdk/java/awt/regtesthelpers/PassFailJFrame.java line 351:
> 
>> 349:      * If you enable screenshots by setting the {@code screenCapture}
>> 350:      * parameter to {@code true}, a <i>Screenshot</i> button is added.
>> 351:      * Clicking the <i>Screenshot</i> button, takes screenshot of
> 
> "screenshots"

And the comma is not needed, it separates the subject from its verb.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21145#discussion_r1774990323


More information about the client-libs-dev mailing list