On Thu, 20 May 2021 10:36:39 GMT, Alexander Zuev <kiz...@openjdk.org> wrote:

>> The @implSpec is part of the specification, it is different from the 
>> @implNote, no?
>> https://bugs.openjdk.java.net/browse/JDK-8266541?focusedCommentId=14419988&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-14419988
>> 
>> If we will specify this method in a way that will require support on all 
>> platforms we will get tck-red immediately after this push.
>
>> The @implSpec is part of the specification, it is different from the 
>> @implNote, no?
>> https://bugs.openjdk.java.net/browse/JDK-8266541?focusedCommentId=14419988&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-14419988
>> 
>> If we will specify this method in a way that will require support on all 
>> platforms we will get tck-red immediately after this push.
> 
> Not exactly. The implNote is a note for future maintainers or people who will 
> extend the functionality of the method. There will be no tck-red because the 
> method is working and we did noted that we are taking into consideration the 
> icon size and whenever technical possible we should return the 
> multiresolution icon. So, for example, on Linux code
> `        FileSystemView fsv = FileSystemView.getFileSystemView();
> 
>         Icon icon = fsv.getSystemIcon(new File("."));
>         Icon icon2 = fsv.getSystemIcon(new File("."), 16);
>         System.out.println("icon = " + icon);
>         System.out.println("icon2 = " + icon2);
> `
> will get icon and icon2 as the same single-resolution icon - but that will 
> change when underlying implementation will be fixed. Right now it is not 
> technical possible to return multi-resolution icon - we do not do it on 
> Linux. Implementing the underlaying code for different system, as i said, is 
> outside of the scope of this change.

The implspec is a specification and we cannot refer to non-public classes in 
that documentation.

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

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

Reply via email to