RFR [11u] JDK-8212118: Clipping problems with complex affine transforms: negative scaling factors or small scaling factors

[David Holmes]

Hi Laurent,

On 12/10/2018 10:50 PM, Laurent Bourgès wrote:
> Hi,
> (this is my first backport request)

 From what I can see this was cleared for backporting by the addition of 
the jdk11u-fix-yes label, as it applied cleanly. All you need to do is 
apply it to the update repo:

http://hg.openjdk.java.net/jdk-updates/jdk11u/

and push (by an authorised committer).

> Please review this patch for OpenJDK 11 updates:
> Backport JBS bug: https://bugs.openjdk.java.net/browse/JDK-8212118

You generally do not need to create a backport issue manually as it will 
be created when you push the change. I edited your backport to remove 
the fix version of 11 as this will be for an 11u version - probably 
11.0.2 - and the 11 might cause a second backport to be created.

Cheers,
David

> Original JBS bug: https://bugs.openjdk.java.net/browse/JDK-8210335
> jdk11u-fix-yes
> <https://bugs.openjdk.java.net/issues/?jql=labels+%3D+jdk11u-fix-yes>
> webrev: http://cr.openjdk.java.net/~lbourges/marlin/marlin-8210335.0/
> 
> *The original patch applied cleanly and jtreg tests are OK:*
> 
> jtreg -retain:all -verbose jdk/jdk11u/test/jdk/sun/java2d/marlin/
> 
> runner starting test: sun/java2d/marlin/Renderer/Test7019861.java
> runner finished test: sun/java2d/marlin/Renderer/Test7019861.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/Renderer/TestNPE.java
> runner finished test: sun/java2d/marlin/Renderer/TestNPE.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/ArrayCacheSizeTest.java
> runner finished test: sun/java2d/marlin/ArrayCacheSizeTest.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/CeilAndFloorTests.java
> runner finished test: sun/java2d/marlin/CeilAndFloorTests.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/CrashNaNTest.java
> runner finished test: sun/java2d/marlin/CrashNaNTest.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/CrashPaintTest.java
> runner finished test: sun/java2d/marlin/CrashPaintTest.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/ClipShapeTest.java
> runner finished test: sun/java2d/marlin/ClipShapeTest.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/CrashTest.java
> runner finished test: sun/java2d/marlin/CrashTest.java
> Error. Test ignored: tests that take a long time and consumes 5Gb memory
> runner starting test: sun/java2d/marlin/DashedRectTest.java
> runner finished test: sun/java2d/marlin/DashedRectTest.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/DashStrokeTest.java
> runner finished test: sun/java2d/marlin/DashStrokeTest.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/JoinMiterTest.java
> runner finished test: sun/java2d/marlin/JoinMiterTest.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/OpenJDKFillBug.java
> runner finished test: sun/java2d/marlin/OpenJDKFillBug.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/ScaleClipTest.java
> runner finished test: sun/java2d/marlin/ScaleClipTest.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/Test7036754.java
> runner finished test: sun/java2d/marlin/Test7036754.java
> Passed. Execution successful
> runner starting test: sun/java2d/marlin/TextClipErrorTest.java
> runner finished test: sun/java2d/marlin/TextClipErrorTest.java
> Passed. Execution successful
> Test results: passed: 14; error: 1
> 
> Regards,
> Laurent
>