On Mon, 17 May 2021 05:08:13 GMT, Alexander Zuev <kiz...@openjdk.org> wrote:

>> src/java.desktop/share/classes/javax/swing/filechooser/FileSystemView.java 
>> line 271:
>> 
>>> 269:     * Example: <pre>
>>> 270:     * FileSystemView fsv = FileSystemView.getFileSystemView();
>>> 271:     * Icon icon = fsv.getSystemIcon("application.exe", 64);
>> 
>> Shouldn't the first parameter be a File instance instead of String?
>> `Icon icon = fsv.getSystemIcon(new File("application.exe"), 64);`
>
> Good catch! Yes, fixed both here and in CSR.

Are we sure that all possible paths can be pointed by the file object? 
Especially some "Windows Libraries" which are accessed by the shell folder?

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

PR: https://git.openjdk.java.net/jdk/pull/2875

Reply via email to