On Thu, 20 May 2021 17:25:33 GMT, Alexander Zuev <kiz...@openjdk.org> wrote:
>> test/jdk/javax/swing/JFileChooser/FileSystemView/SystemIconTest.java line 72: >> >>> 70: if (icon.getImage() instanceof MultiResolutionImage) { >>> 71: MultiResolutionImage mri = (MultiResolutionImage) >>> icon.getImage(); >>> 72: if (mri.getResolutionVariant(size, size) == null) { >> >> This is to describe one of my questions above, is this instanceof+cast >> cannot be improved? Why we cannot always wrap the data in the MRI and if we >> have only one icon return the MRI with one resolution? > >> This is to describe one of my questions above, is this instanceof+cast >> cannot be improved? Why we cannot always wrap the data in the MRI and if we >> have only one icon return the MRI with one resolution? > > No because in specification we say that we do not guarantee the the icon > returned will be MRI. We know how it works on Windows so we test it this way. Here I am not talking about specification, I am talking about the usage of it, is the instanceof+cast is helpful? It does not look good, why we cannot always return MRI? ------------- PR: https://git.openjdk.java.net/jdk/pull/2875