Merge pull request #12 from FabioVassallo/nan_issue

Workaround to avoid calls to contentStream.moveTo with NaN arguments.
rototor · Apr 11, 2018 · ae5bdb4 · ae5bdb4
2 parents 8bf089c + ed5fb99
commit ae5bdb4
Showing 1 changed file with 19 additions and 16 deletions.
diff --git a/src/main/java/de/rototor/pdfbox/graphics2d/PdfBoxGraphics2D.java b/src/main/java/de/rototor/pdfbox/graphics2d/PdfBoxGraphics2D.java
@@ -829,22 +829,25 @@ private void walkShape(Shape clip) throws IOException {
 		PathIterator pi = clip.getPathIterator(tf);
 		float[] coords = new float[6];
 		while (!pi.isDone()) {
-			switch (pi.currentSegment(coords)) {
-			case PathIterator.SEG_MOVETO:
-				contentStream.moveTo(coords[0], coords[1]);
-				break;
-			case PathIterator.SEG_LINETO:
-				contentStream.lineTo(coords[0], coords[1]);
-				break;
-			case PathIterator.SEG_QUADTO:
-				contentStream.curveTo1(coords[0], coords[1], coords[2], coords[3]);
-				break;
-			case PathIterator.SEG_CUBICTO:
-				contentStream.curveTo(coords[0], coords[1], coords[2], coords[3], coords[4], coords[5]);
-				break;
-			case PathIterator.SEG_CLOSE:
-				contentStream.closePath();
-				break;
+			int segment = pi.currentSegment(coords);
+			if(Float.isFinite(coords[0])) {
+				switch (segment) {
+				case PathIterator.SEG_MOVETO:
+					contentStream.moveTo(coords[0], coords[1]);
+					break;
+				case PathIterator.SEG_LINETO:
+					contentStream.lineTo(coords[0], coords[1]);
+					break;
+				case PathIterator.SEG_QUADTO:
+					contentStream.curveTo1(coords[0], coords[1], coords[2], coords[3]);
+					break;
+				case PathIterator.SEG_CUBICTO:
+					contentStream.curveTo(coords[0], coords[1], coords[2], coords[3], coords[4], coords[5]);
+					break;
+				case PathIterator.SEG_CLOSE:
+					contentStream.closePath();
+					break;
+				}
 			}
 			pi.next();
 		}