Jim,
> Test program, line 490 - MOVETO has 2 coordinates associated with it. >> > > Well spotted: I did it too quickly, sorry. > > Test program, line 492 - perhaps we should throw an exception on default >> since it indicates a problem with the iterator >> > > Ok. > Here is the new webrev: http://cr.openjdk.java.net/~lbourges/path2D/Path2D.4/ I fixed the getLength(type) method: 482 static int getLength(int type) { 483 switch(type) { 484 case PathIterator.SEG_CUBICTO: 485 return 6; 486 case PathIterator.SEG_QUADTO: 487 return 4;* 488 case PathIterator.SEG_LINETO: 489 case PathIterator.SEG_MOVETO: 490 return 2; * 491 case PathIterator.SEG_CLOSE: 492 return 0;* 493 default: 494 throw new IllegalStateException("Invalid type: " + type); * 495 } 496 } If it is good, could you push this patch in both graphics-rasterizer and java2d repositories ? or do you prefer I push this patch into graphics-rasterizer and later it will be merged into java2d and possibly backported to JDK8 ? Laurent
