Build changes still look good.
/Erik
On 2015-02-16 12:57, Sergey Bylokhov wrote:
Hello,
Here is an updated version of the fix:
http://cr.openjdk.java.net/~serb/8039269/webrev.01
----- alan.bate...@oracle.com wrote:
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
