Review request for 8169294:

Kevin Rushforth kevin.rushforth at oracle.com
Mon Nov 7 22:50:12 UTC 2016


I'll review the test tomorrow (I'm a little backed up on my code reviews 
given the impending change to build with jigsaw JDK).

-- Kevin


Laurent Bourgès wrote:
> Jim,
>
> 2016-11-07 21:05 GMT+01:00 Jim Graham <james.graham at oracle.com 
> <mailto:james.graham at oracle.com>>:
>
>     I'd like to see Kevin review the test as I'm not the best expert
>     on our JUnit framework.
>
>
> I just added @Test annotations and kept the jtreg tags in header (for 
> information).
> I could add asserts but JUnit does intercept any thrown exception and 
> reports a test failure in such case.
>
> I managed to undestand how to run my test with gradle:
> gradle :graphics:test  --tests test.com.sun.javafx.geom.Path2DGrowTest
>
> It is passing and the reports indicates that the 2 new tests are OK 
> (report in modules/javafx.graphics/build/reports/tests/index.html)
>  
>
>
>     It looks like it is mostly just going to emit some printouts about
>     performance (using echo() and log()) and verify that we don't get
>     any ArrayBounds related exceptions (or worse, OOME)?
>
>
> Exactly, it is passing as there is no runtime exception and the 
> performance issue is fixed as indicated by the few logged lines.
>  
>
>
>     The only difference in Path2D.java I noted is that the Java2D
>     version has an EXPAND_MIN which is 10, but you re-use INIT_SIZE,
>     which is 20, here for the same purpose.
>
>
> You're right; I think I didn't want to add an extra constant but if 
> you prefer being more consistent, I can prepare another webrev.
>
> Laurent


More information about the openjfx-dev mailing list