Re: [OpenJDK 2D-Dev] JDK9: RFR: 8039342: Fix raw and unchecked warnings in sun.awt.*

2014-04-07 Thread Henry Jen
Thanks Joe for pointing out the request should go to awt-dev. Cheers, Henry On 04/07/2014 04:13 PM, Joe Darcy wrote: Hi Henry, Thanks for looking into this. FWIW, while I was working on JDK-8039109: Fix unchecked and raw lint warnings in java.awt I started looking at some sun.awt.*

Re: [OpenJDK 2D-Dev] JDK9: RFR: 8039342: Fix raw and unchecked warnings in sun.awt.*

2014-04-07 Thread Henry Jen
Thanks Joe for pointing out the review request ned to go to awt-dev. Cheers, Henry On 04/07/2014 04:13 PM, Joe Darcy wrote: Hi Henry, Thanks for looking into this. FWIW, while I was working on JDK-8039109: Fix unchecked and raw lint warnings in java.awt I started looking at some su

Re: [OpenJDK 2D-Dev] JDK9: RFR: 8039342: Fix raw and unchecked warnings in sun.awt.*

2014-04-07 Thread Joe Darcy
Hi Henry, Thanks for looking into this. FWIW, while I was working on JDK-8039109: Fix unchecked and raw lint warnings in java.awt I started looking at some sun.awt.* type too and I noticed the same problem in AreaOp.pruneEdges(). The comment I added to my in-progress webrev is: 203

[OpenJDK 2D-Dev] JDK9: RFR: 8039342: Fix raw and unchecked warnings in sun.awt.*

2014-04-07 Thread Henry Jen
Hi, Please review the webrev cleans up raw and unchecked warnings in sun.awt, http://cr.openjdk.java.net/~henryjen/jdk9/8039342/0/webrev/ The following changes in AreaOp::pruneEdges() is particular worth attention, when numedges < 2, two different type are mixed up in the past with use of raw

Re: [OpenJDK 2D-Dev] RFR: JDK8u20 backport of 8035569: JNI correctness fixes in WPrinterJob.cpp

2014-04-07 Thread Sergey Bylokhov
Hi, Phil. Looks fine. On 07.04.2014 1:31, Phil Race wrote: On 4/5/14 8:14 AM, Sergey Bylokhov wrote: Hi, Phil. Looks like webrev/jbslink for JDK8u20 are for other fix. I think that's because I hand-edited the URL in thunderbird and it changed the visible text but not the actual link :-( So th