On Thu, 20 May 2021 08:13:21 GMT, Alexander Zuev <kiz...@openjdk.org> wrote:

>> src/java.desktop/share/classes/javax/swing/filechooser/FileSystemView.java 
>> line 272:
>> 
>>> 270:     * returned is a multi-resolution icon image,
>>> 271:     * which allows better support for High DPI environments
>>> 272:     * with different scaling factors.
>> 
>> Is the above text correct on all platforms? If it is not always MRI then how 
>> the user should use the icon? instanceof+cast? BTW an example does not show 
>> how to solve the bug itself, on how to access the "large icons".
>> 
>> Need to clarify: the implSpec is a part of the specification so can we point 
>> the non public "ShellFolder" class?
>
> implSpec marks that the paragraph below describes the details and logic of 
> the default implementation and not the API specification. This tag also says 
> that it can be changed in overriding or extending methods so it is Ok to 
> specify non-public class to help describe the implementation specifics.
> 
> As for the correctness on all platforms - that's the end goal of this new 
> method and i believe it should be implemented this way everywhere where 
> technically possible. But exact implementation on all platforms except 
> Windows is outside of the scope of this exact changeset.

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.

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

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

Reply via email to