On Tue, 11 Oct 2022 07:02:18 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> Abhishek Kumar has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Review comment fix and test case changed to manual
>
> src/java.desktop/share/classes/com/sun/java/swing/plaf/gtk/GTKFileChooserUI.java
>  line 997:
> 
>> 995:             File crntDir = getFileChooser().getCurrentDirectory();
>> 996:             if (crntDir != null && crntDir.getParentFile() == null &&
>> 997:                 files.contains(new File("/.."))) {
> 
> Is it not possible to use "FileSystemView.isFileSystemRoot(File)" for this?

I have not checked this.
Do you mean to check `FileSystemView.isFileSystemRoot(crntDir )` ?

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

PR: https://git.openjdk.org/jdk/pull/10390

Reply via email to