On 13/02/2015 18:01, Sergey Bylokhov wrote:
Hello.
Please review the fix for jdk 9.
As requested cursor related properties/images were moved from /lib to the 
java.desktop.
  - Image prefixes were removed because I moved them to the os specific 
location.
  - Windows version of cursors.properties was removed, because it was mostly 
the same as shared version.
  - ***NoDrop images were removed, because they are the same as 
invalid32x32.gif. cursors.properties was updated accordingly.
  - Note that if in the future some osx/unix will demand some specific 
images/properties, it will be necessary to place them to the correct folder 
only.

Bug: https://bugs.openjdk.java.net/browse/JDK-8039269
Webrevs can be found at:
     http://cr.openjdk.java.net/~serb/8039269/webrev.00/root
     http://cr.openjdk.java.net/~serb/8039269/webrev.00/jdk

As Mandy pointed, you use try-with-resources. Also I think you can use "in" rather than "fis" as the input stream is no longer a FileInputStream.

I think the main thing here is that it has been established that cursor.properties is not a supported interface, that was the question that issue was trying to establish for a long time. Also good to have these files moved out of the conf directory.

-Alan

Reply via email to