[OpenJDK 2D-Dev] RFR : 8049893 Replace uses of 'new Integer()' with appropriate alternative across client classes

2014-07-10 Thread Otávio Gonçalves de Santana
https://bugs.openjdk.java.net/browse/JDK-8049893 http://cr.openjdk.java.net/~prr/8049893/ -- Cheers!. Otávio Gonçalves de Santana blog: http://otaviosantana.blogspot.com.br/ twitter: http://twitter.com/otaviojava site: *http://about.me/otaviojava * 55 (11) 98

[OpenJDK 2D-Dev] RFR: 8u backport of 7067052 : Default printer media is ignored

2014-07-10 Thread Phil Race
Jennifer, Please review/check http://cr.openjdk.java.net/~prr/7067052.8u/ This is a 100% identical backport of your JDK 9 fix https://bugs.openjdk.java.net/browse/JDK-7067052 http://hg.openjdk.java.net/jdk9/client/jdk/rev/327e816d4c7e It needs to be backported as part of a series of fixes for F

Re: [OpenJDK 2D-Dev] [9] Review Request for 8037511: tidy warnings cleanup for java.awt - 2d part

2014-07-10 Thread Phil Race
This all looks OK javadoc-wise. Although I am quite sure you cannot copyright simply removing a element so updating the copyright year was entirely a waste of time .. and not really correct either. However given that people have annually run scripts that update the (c) string in any file up

Re: [OpenJDK 2D-Dev] [9] Review request for 8041460, PNGImageWriter creates metadata with incorrect bitDepth

2014-07-10 Thread Andrew Brygin
The updated fix looks fine to me. Thanks, Andrew On 7/9/2014 10:44 PM, Phil Race wrote: This now looks fine. Since the fix changed since Andrew approved, make sure you get a 2nd reviewer. Also the putback comment must use bug ID 4991647 -phil. On 7/9/2014 9:53 AM, mikhail cherkasov wrote: H