On Thu, 21 Oct 2021 04:24:11 GMT, Prasanta Sadhukhan <[email protected]>
wrote:
>> Alexander Zuev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Call dispose() from the test final block. Also added original exception to
>> the unexpected exception handler for easier triage in the future.
>
> test/jdk/java/awt/Mouse/GetMousePositionTest/GetMousePositionWithOverlay.java
> line 52:
>
>> 50: doTest();
>> 51: } catch (Exception e) {
>> 52: if (e instanceof RuntimeException) {
>
> Do we need this catch block? I guess main() throws Exception so we can just
> have try-finally block, no?
The catch block was already there and i decided not to remove it and moved the
rest of the code instead of the try block.
-------------
PR: https://git.openjdk.java.net/jdk/pull/6028
