<Swing Dev> [11] RFR JDK-8198004: javax/swing/JFileChooser/6868611/bug6868611.java throws error

Jayathirth D V jayathirth.d.v at oracle.com
Thu Feb 22 06:04:32 UTC 2018


Hi Semyon,

 

Thanks for your inputs.

I have moved file deletion logic to finally block.

 

Please find updated webrev for review:

http://cr.openjdk.java.net/~jdv/8198004/webrev.01/ 

 

Thanks,

Jay

 

From: Semyon Sadetsky 
Sent: Thursday, February 22, 2018 7:03 AM
To: Jayathirth D V; swing-dev at openjdk.java.net
Subject: Re: <Swing Dev> [11] RFR JDK-8198004: javax/swing/JFileChooser/6868611/bug6868611.java throws error

 

Hi Jay,

Please delete files in finally block to avoid files stay undeleted in case of exception.

--Semyon

 

 

On 02/21/2018 03:09 AM, Jayathirth D V wrote:

Hello All,

 

Please review the following fix in JDK11 :

 

Bug : https://bugs.openjdk.java.net/browse/JDK-8198004 

Webrev : HYPERLINK "http://cr.openjdk.java.net/%7Ejdv/8198004/webrev.00/"http://cr.openjdk.java.net/~jdv/8198004/webrev.00/ 

 

Issue: If we run javax/swing/JFileChooser/6868611/bug6868611.java it times out and throws jtreg error.

 

Root cause : In the test case we create 1000 files in default temporary directory and then call FileSystemView.getFiles() on default temporary directory. But if the same temporary directory has been used previously and if contains lot of unnecessary files FileSystemView.getFiles() will take lot of time. My machine temp folder had lot of files and after I delete major amount of files from temp directory and run the test again it passes without any timeout.

 

Solution: We should not use default temporary directory for creating 1000 files and call FileSystemView.getFiles() on same temp folder. Creating sub-folder under default temp directory and using it for test case resolves the issue.

 

Thanks,

Jay

 

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20180221/791ff15c/attachment.html>


More information about the swing-dev mailing list