[OpenJDK 2D-Dev] RFR JDK-8184429: Path clipper added in Marlin2D & MarlinFX 0.8.0

Kevin Rushforth kevin.rushforth at oracle.com
Mon Dec 11 17:13:06 UTC 2017


I started looking the FX patch on Friday. I hope to finish my review today.

-- Kevin


Philip Race wrote:
> Yes, I approved the 2D patch. I have not yet looked at the FX version.
>
> -phil
>
> On 12/10/17, 12:39 PM, Laurent Bourgès wrote:
>> Phil,
>>
>> Thanks for your review.
>>
>> I suppose you approved the Java2D patch for the RFR thread: [10] RFR 
>> JDK-8191814:
>> http://mail.openjdk.java.net/pipermail/2d-dev/2017-November/008741.html
>>
>> I will push tomorrow to jdk forrest as phil & sergey? approved 2D 
>> changes.
>>
>>
>> This RFR thread remains open for the JavaFX patch that Kevin started 
>> to test & review.
>>
>> Cheers,
>> Laurent
>>
>> 2017-12-10 18:36 GMT+01:00 Philip Race <philip.race at oracle.com 
>> <mailto:philip.race at oracle.com>>:
>>
>>     I'm giving this an OK.
>>     I've looked at the code, if not the maths, and run our regression
>>     test suite.
>>     I had a bit of trouble with my full baseline run against which to
>>     compare so I've
>>     re-run just the test failures that seem like they might go
>>     anywhere need this code.
>>     and they were pre-existing.
>>
>>     -phil.
>>
>>
>>     On 11/16/17, 3:01 PM, Laurent Bourgès wrote:
>>>     Phil,
>>>
>>>     Here is an updated webrev:
>>>     http://cr.openjdk.java.net/~lbourges/marlin/marlin-082-8184429.1/
>>>     <http://cr.openjdk.java.net/%7Elbourges/marlin/marlin-082-8184429.1/>
>>>
>>>     It fixes the ClipShapeTest to run ~35s (< 2mins) on my latop: 22
>>>     test setups only (5000 random polygons each) that covers all
>>>     aspects of the new clipping algorithm.
>>>
>>>     I wonder if I should remove the 'slow' mode that has till @ignore:
>>>     both tests are ignored if I run jtreg -ignore:quiet although I
>>>     would like to only ignore the @ignore run (1/2).
>>>
>>>
>>>     Once again, I have to create a new JBS bug for Marlin2D tomorrow
>>>     then start a new RFR with a complete change log.
>>>
>>>     Laurent
>>>
>>>     2017-11-16 10:50 GMT+01:00 Laurent Bourgès
>>>     <bourges.laurent at gmail.com <mailto:bourges.laurent at gmail.com>>:
>>>
>>>         Sergey,
>>>
>>>             You can generate a number of images using a few threads
>>>             when the flag is on then switch it off and compares
>>>             results. The test should not draw different(on/off)
>>>             modes in parallel but it can draw the images for the
>>>             same mode.
>>>
>>>
>>>         Yes that is always possible but I do not want to spend too
>>>         much time improving the test to parallelize it.
>>>
>>>         I fixed it last night and it runs only 22 test setups (1
>>>         stroke width = 8px, no dashes) and takes ~ 30 seconds on my
>>>         laptop (single-thread).
>>>         I verified that any bug in Marlin clipping is detected by
>>>         this new variant of the tests.
>>>         Will provide a new patch asap.
>>>          
>>>
>>>
>>>                 Finally I will minimize the number of stroker tests:
>>>                 use only 1 stroke width (=5px) and that should be
>>>                 enough to stay below timeout (2mins).
>>>
>>>
>>>             Note that the test can be run on some slow/virtual
>>>             systems, it would be good to have some additional time.
>>>
>>>                 Is there any documentation about jtreg tags ?
>>>
>>>
>>>             You can find it here:
>>>             http://openjdk.java.net/jtreg/tag-spec.html
>>>             <http://openjdk.java.net/jtreg/tag-spec.html>
>>>
>>>
>>>         Thanks.
>>>
>>>         Laurent
>>>
>>>
>>>
>>>
>>>     -- 
>>>     -- 
>>>     Laurent Bourgès
>>
>>
>>
>>
>> -- 
>> -- 
>> Laurent Bourgès
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20171211/8d46fb6c/attachment-0001.html>


More information about the 2d-dev mailing list