Re: [OpenJDK 2D-Dev] Review request for http://cr.openjdk.java.net/~ceisserer/fix10/

2013-09-26 Thread Clemens Eisserer
Hi Phil, the webrev and test need a little clean-up : adding the @test and @bug tag + GPL Please find a corrected version at http://cr.openjdk.java.net/~ceisserer/8024895/webrev.01/ along with a (hopefully) jtreg conformant testcase at

Re: [OpenJDK 2D-Dev] Review request for http://cr.openjdk.java.net/~ceisserer/fix10/

2013-09-26 Thread Phil Race
Looks OK. I'll push once a 2nd set of eyes concurs. -phil. On 9/26/13 12:04 PM, Clemens Eisserer wrote: Hi Phil, the webrev and test need a little clean-up : adding the @test and @bug tag + GPL Please find a corrected version at http://cr.openjdk.java.net/~ceisserer/8024895/webrev.01/ along

Re: [OpenJDK 2D-Dev] Review request for http://cr.openjdk.java.net/~ceisserer/fix10/

2013-09-16 Thread Phil Race
Hi Clemens, I created 8024895: xrender MaskImage cache isn't accounting for change in alpha the webrev and test need a little clean-up : adding the @test and @bug tag + GPL There's a minor formatting error here :- int currentEA =xrMgr.getAlphaColor().getAlpha(); .. missing space after =

[OpenJDK 2D-Dev] Review request for http://cr.openjdk.java.net/~ceisserer/fix10/

2013-09-14 Thread Clemens Eisserer
Hi, Please review my fix at http://cr.openjdk.java.net/~ceisserer/fix10/ I haven't created a an official bug-id, because due to the migration to JIRA those user reports don't seem to be directly useable anyway. Problem description: For transformed blits the xrender pipeline uses caching-logic to