[9] Review Request for 8051636: DataTransferer optional dependency on RMI

2015-06-16 Thread Semyon Sadetsky
Hello, Please review fix for JDK9-JIGSAW: bug: https://bugs.openjdk.java.net/browse/JDK-8051636 webrev: http://cr.openjdk.java.net/~ssadetsky/8051636/webrev.00/ I have found only one run-time dependency on java.rmi module in sun.datatransfer.DataFlavorUtil.RMI class. Now it checks module read

Re: [OpenJDK 2D-Dev] [9] Review Request: 8080993 Compilation errors with recent clang in awt_parseImage.c and splashscreen_sys.m

2015-06-16 Thread Sergey Bylokhov
Hello, Any volunteers to review the fix? On 08.06.15 19:52, Sergey Bylokhov wrote: Hello. Please review the small fix for jdk9. Two simple warnings were fixed. The fix contributed by Staffan Larsen. Bug: https://bugs.openjdk.java.net/browse/JDK-8080993 Webrev can be found at: http://cr.openjdk

Re: [OpenJDK 2D-Dev] [9] Review Request: 8014241 Clean up unused JNI fields and methods in imageInitIDs.h

2015-06-16 Thread Sergey Bylokhov
Hello, Any volunteers to review the fix? On 10.06.15 20:51, Sergey Bylokhov wrote: Hello. Please review a small cleanup for jdk9. Unused fields/methods were removed. Bug: https://bugs.openjdk.java.net/browse/JDK-8014241 Webrev can be found at: http://cr.openjdk.java.net/~serb/8014241/webrev.

Re: [9] Review Request: 8025492 Hand cursor does not use Windows' system cursor

2015-06-16 Thread Sergey Bylokhov
Hello, Any volunteers to be a second reviewer? On 11.06.15 19:52, Anton V. Tarasov wrote: Looks fine to me. Anton. On 09.06.2015 20:03, Sergey Bylokhov wrote: Hello. Please review the small fix for jdk9. The fix was contributed-by morvan.lemes...@gmail.com [1] For historical reasons jdk uses

Re: [9] Review Request: JDK-8080932 [TEST_BUG] Test java/awt/BasicStroke/DashStrokeTest.java fails with Bad script error due to improper @run notation

2015-06-16 Thread pooja chopra
Hi All, Gentle reminder .Please review below fix . Regards, Pooja On 6/4/2015 8:18 PM, Alexander Scherbatiy wrote: The fix looks good to me. Thanks, Alexandr. On 6/3/2015 3:50 PM, pooja chopra wrote: Hi All , Please review below fix. Regards, Pooja On 5/22/2015 7:23 PM, pooja chopra wro

Re: [9] Review Request: JDK-8080953 [TEST_BUG]Test java/awt/FontClass/DebugFonts.java fails due to wrongly typed bugid

2015-06-16 Thread pooja chopra
Hi All , Gentle Reminder. Please review below fix . Regards, Pooja On 5/22/2015 10:34 PM, Sergey Bylokhov wrote: cc 2d On 22.05.15 19:59, pooja chopra wrote: Hello, Please review a fix for issue: 8080953 [TEST_BUG]Test java/awt/FontClass/DebugFonts.java fails due to wrongly typed bugid Test bu