On Thu, 3 Oct 2024 07:40:15 GMT, Abhishek Kumar <[email protected]> wrote:

>> Jayathirth D V has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Update based on review comments
>
> test/jdk/java/awt/Window/InvalidFocusLostEventTest/InvalidFocusLostEventTest.java
>  line 60:
> 
>> 58:             countDownLatch.await();
>> 59:             if (fm.getFocusOwner() != b) {
>> 60:                 failed = true;
> 
> I guess exception can be thrown here and then no need of `failed` variable 
> and checking from finally block can be removed.

Good idea. Removed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21281#discussion_r1785812742

Reply via email to