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

2015-07-06 Thread pooja chopra
Thanks. On 7/2/2015 5:18 PM, Alexander Zvegintsev wrote: Hi, you already have two reviewers, it is sufficient to push. Anyway, the fix looks good to me. Thanks, Alexander. On 07/02/2015 10:28 AM, pooja chopra wrote: Hi All, Please review below fix. Regards, Pooja On 6/17/2015 12:05 PM,

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

2015-07-06 Thread pooja chopra
Thanks. On 7/2/2015 5:13 PM, Sergey Bylokhov wrote: +1 On 02.07.15 14:41, Alexander Zvegintsev wrote: Looks fine to me. Thanks, Alexander. On 07/02/2015 10:29 AM, pooja chopra wrote: Hi All, Please review below fix. Regards, Pooja On 6/17/2015 12:06 PM, pooja chopra wrote: Hi All ,

AWT Dev Awt Dev [9] Review Request for 8130242: DataFlavorComparator transitivity exception

2015-07-06 Thread Semyon Sadetsky
Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8130242 webrev: http://cr.openjdk.java.net/~ssadetsky/8130242/webrev.00/ Data transfer's flavor comparator violates transitivity. --Semyon

Re: AWT Dev [9] Review request for 8081787: [macosx] MalformedURLException is thrown during reading data for application/x-java-url; class=java.net.URL flavor

2015-07-06 Thread mikhail cherkasov
Hi Sergey, UnsupportedFlavorException is thrown because clipboard has something that can't be translated to application/x-java-url. You should copy link from Safari or copy file from finder, only then you will see MalformedURLException. So testing requires manual actions, because we need a

AWT Dev [9] Review Request: 8130525 Build failure on jdk9-client solaris-sparcv9

2015-07-06 Thread Sergey Bylokhov
Hello. Please review the fix for jdk9. In the fix for JDK-7188942 I forgot to cleanup the mapfiles when I removed initPbuffer() method. Bug: https://bugs.openjdk.java.net/browse/JDK-8130525 Webrev can be found at: http://cr.openjdk.java.net/~serb/8130525/webrev.00 -- Best regards, Sergey.

Re: AWT Dev [9] Review Request: 8130525 Build failure on jdk9-client solaris-sparcv9

2015-07-06 Thread Alexander Zvegintsev
Looks fine to me. Thanks, Alexander. On 07/06/2015 04:20 PM, Sergey Bylokhov wrote: Hello. Please review the fix for jdk9. In the fix for JDK-7188942 I forgot to cleanup the mapfiles when I removed initPbuffer() method. Bug: https://bugs.openjdk.java.net/browse/JDK-8130525 Webrev can be

Re: AWT Dev Awt Dev [9] Review Request for 8130242: DataFlavorComparator transitivity exception

2015-07-06 Thread Alexander Scherbatiy
On 7/6/2015 2:17 PM, Semyon Sadetsky wrote: Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8130242 webrev: http://cr.openjdk.java.net/~ssadetsky/8130242/webrev.00/ Data transfer's flavor comparator violates transitivity. Could you give an example of

Re: AWT Dev Awt Dev [9] Review Request for 8130242: DataFlavorComparator transitivity exception

2015-07-06 Thread Semyon Sadetsky
Transitivity (2) violated: X: java.awt.datatransfer.DataFlavor[mimetype=text/uri-list;representationclass=java.nio.ByteBuffer;charset=UTF-8], Y: java.awt.datatransfer.DataFlavor[mimetype=application/x-java-text-encoding;representationclass=java.io.InputStream], Z: