RFR: JDK-8316741: BasicStroke.createStrokedShape miter-limits failing on small shapes
Damon Nguyen
dnguyen at openjdk.org
Fri Oct 20 18:47:37 UTC 2023
On Sun, 1 Oct 2023 20:33:54 GMT, Laurent Bourgès <lbourges at openjdk.org> wrote:
> For stroked shapes (no Renderer use), the min-join-error check is disabled.
> Added new jtreg test to verify join using pixel counts
Changes look reasonable and the test looks good. Ran it myself and it passes with the changes.
test/jdk/sun/java2d/marlin/TestCreateStrokedShapeJoins.java line 39:
> 37: */
> 38: public class TestCreateStrokedShapeJoins {
> 39:
Suggestion:
/*
* @test
* @bug 8316741
* @summary Verifies that Marlin renderer's Stroker generates properly joins
* in createStrokedShape()
* @run main TestCreateStrokedShapeJoins
*/
public class TestCreateStrokedShapeJoins {
test/jdk/sun/java2d/marlin/TestCreateStrokedShapeJoins.java line 44:
> 42: private final static int W = 200;
> 43:
> 44: private final static int[] REF_COUNTS = new int[]{4561, 4790, 5499};
Suggestion:
private final static int[] REF_COUNTS = new int[] {4561, 4790, 5499};
test/jdk/sun/java2d/marlin/TestCreateStrokedShapeJoins.java line 47:
> 45:
> 46: public static void main(String[] args) throws Exception {
> 47: final int[] test = new int[]{
Suggestion:
final int[] test = new int[] {
-------------
Marked as reviewed by dnguyen (Committer).
PR Review: https://git.openjdk.org/jdk/pull/16002#pullrequestreview-1690675540
PR Review Comment: https://git.openjdk.org/jdk/pull/16002#discussion_r1367348124
PR Review Comment: https://git.openjdk.org/jdk/pull/16002#discussion_r1367379410
PR Review Comment: https://git.openjdk.org/jdk/pull/16002#discussion_r1367379573
More information about the client-libs-dev
mailing list