Re: [OpenJDK 2D-Dev] [8] Review request for 8023990: regression : postscript file size increase from 6u17

2013-12-05 Thread Andrew Brygin
Looks fine. Thanks, Andrew On 11/22/2013 7:24 PM, anton nashatyrev wrote: Hello, this is a duplicate request: could you please review the following fix: fix: http://cr.openjdk.java.net/~alitvinov/8023990/webrev.00 http://cr.openjdk.java.net/%7Ealitvinov/8023990/webrev.00 bug:

Re: [OpenJDK 2D-Dev] AWT Dev [8] Review request for 8011059 [macosx] Make JDK demos look perfect on retina displays

2013-12-05 Thread Sergey Bylokhov
Hi, Alexander. Count me as a second reviewer. On 12/4/13 10:16 PM, Jim Graham wrote: Hi Alexander, It looks good to go. I only skimmed the other parts of the fix on the assumption that they haven't changed in a few revisions, but it all looked good. Glad to see you fixed the dimension

Re: [OpenJDK 2D-Dev] [2d dev] [8] Review request for JDK-8028272 : Tidy warnings cleanup for javax.print package

2013-12-05 Thread Phil Race
Hi, I am not happy with approving this until I have had a chance to review 1) The actual warnings from tidy 2) The effect on the generated javadoc. I think P adds extra space that br does not, so don't jump to conclusions about what the original author was trying to achieve. -phil. On

Re: [OpenJDK 2D-Dev] AWT Dev [8] Review request for 8011059 [macosx] Make JDK demos look perfect on retina displays

2013-12-05 Thread Mike Swingler
On Dec 3, 2013, at 10:10 AM, Jim Graham james.gra...@oracle.com wrote: Hi Mike, One more question about @2x handling on MacOS. Clearly, in the simple case of someone loading an ordinary foo.png and painting it on a retina display without doing anything special with the transform, it

[OpenJDK 2D-Dev] RFR: 8029628: Large amount of graphic artifacts

2013-12-05 Thread Vadim Pakhnushev
Hello, Please review a fix for the https://bugs.openjdk.java.net/browse/JDK-8029628 http://cr.openjdk.java.net/~vadim/8029628/webrev.00/ Intel HD Graphics 2000/3000 seems to be not usable (yet again!) So the proposed fix is to blacklist all Intel cards except for HD 4000+, which seems to be