On Tue, 16 Feb 2021 16:06:27 GMT, Alexandre Iline <[email protected]>
wrote:
>> Amresh Sahu has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Update bug6698013.java
>>
>> Removed duplicate code.
>
> test/jdk/javax/swing/JFileChooser/6698013/bug6698013.java line 50:
>
>> 48: SwingUtilities.invokeAndWait(() -> new bug6698013().init());
>> 49: } catch (InvocationTargetException | InterruptedException e) {
>> 50: e.printStackTrace();
>
> The test should, probably, fail in case of the exception.
Yes, it's safe to declare `main` with `throws Exception`; `main` is for
convenience to run the test quickly on the command line.
The test harness does not use `main`, it runs the test as applet.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2587