[OpenJDK 2D-Dev] [100303] Graphics: custom Composite + anti-aliasing = RasterFormatException

2013-03-05 Thread Matthias Wienand
Hi, when drawing outside of the client area while using a custom Composite when anti-aliasing is enabled, then a RasterFormatException is thrown. When anti-aliasing is disabled, the behavior is as expected, i.e. the drawing is correctly clipped and the custom Composite is correctly applied. I open

Re: [OpenJDK 2D-Dev] [8] request for review: 7152608: [macosx] Crash in liblwawt.dylib in AccelGlyphCache_RemoveCellInfo

2013-03-05 Thread Anton V. Tarasov
Hi Andrew, The changes look fine for me in general (though, I'm not an expert in this area). Thanks, Anton. On 04.03.2013 20:04, Andrew Brygin wrote: Hello, could you please review forward port of the fix for 7152608 to jdk8? Bug: http://bugs.sun.com/view_bug.do?bug_id=7152608 Webrev: http:

[OpenJDK 2D-Dev] hg: jdk8/2d/jdk: 7152608: [macosx] Crash in liblwawt.dylib in AccelGlyphCache_RemoveCellInfo

2013-03-05 Thread andrew . brygin
Changeset: f3368a3fc023 Author:bae Date: 2013-03-05 17:18 +0400 URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/f3368a3fc023 7152608: [macosx] Crash in liblwawt.dylib in AccelGlyphCache_RemoveCellInfo Reviewed-by: jgodinez, ant ! src/macosx/classes/sun/font/CStrike.java ! src/macos

Re: [OpenJDK 2D-Dev] [8] Review request for 8007295: Reduce number of warnings in awt classes

2013-03-05 Thread Andrew Brygin
I have reviewed changes in src/share/classes/java/awt/RenderingHints.java and src/share/classes/java/awt/geom/AffineTransform.java, and they look good to me. Thanks, Andrew On 3/5/2013 2:50 PM, Anthony Petrov wrote: Looks good to me. There's also some 2D code, so I'm adding 2d-dev@ for an ad

Re: [OpenJDK 2D-Dev] [8] Review request for 8007295: Reduce number of warnings in awt classes

2013-03-05 Thread Anthony Petrov
Looks good to me. There's also some 2D code, so I'm adding 2d-dev@ for an additional review. -- best regards, Anthony On 3/4/2013 20:15, mikhail cherkasov wrote: Hi all, Please review the following changes: Webrev: http://cr.openjdk.java.net/~mcherkas/8007295/webrev.00/