RFR: 8354493: Opensource Several MultiScreen and Insets related tests [v2]

Abhishek Kumar abhiscxk at openjdk.org
Wed Apr 16 06:17:27 UTC 2025


On Tue, 15 Apr 2025 20:18:49 GMT, Hendrik Schick <duke at openjdk.org> wrote:

>> Abhishek Kumar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Limit to 80 cols
>
> test/jdk/java/awt/Multiscreen/DialogTest.java line 210:
> 
>> 208:                 }
>> 209:             } else {
>> 210:                 if (frameOwned) {
> 
> this code part is in 'if' and 'else' path, can be done before.

No.. there is a difference.
One is for `JDialog` and other is for `Dialog`

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24654#discussion_r2046152978


More information about the client-libs-dev mailing list