PDFBOX-5660: optimize, as suggested by Valery Bokov; closes #447

THausherr · THausherr · commit 77be3ecbd8c5 · 2026-05-10T11:10:04.000Z
git-svn-id: https://svn.apache.org/repos/asf/pdfbox/trunk@1934043 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/pdfbox/src/main/java/org/apache/pdfbox/printing/PDFPrintable.java b/pdfbox/src/main/java/org/apache/pdfbox/printing/PDFPrintable.java
@@ -219,8 +219,9 @@ public int print(Graphics graphics, PageFormat pageFormat, int pageIndex)
             float rasterDpi = dpi;
             if (rasterDpi == RASTERIZE_DPI_AUTO)
             {
-                rasterDpi = new Matrix(graphics2D.getTransform()).getScalingFactorX() * 72.0f;
-                LOG.debug("auto raster dpi: {}, g2d: {}, g2d transform: {}", rasterDpi, graphics2D, graphics2D.getTransform());
+                AffineTransform transform = graphics2D.getTransform();
+                rasterDpi = new Matrix(transform).getScalingFactorX() * 72.0f;
+                LOG.debug("auto raster dpi: {}, g2d: {}, g2d transform: {}", rasterDpi, graphics2D, transform);
             }
 
             PDPage page = pageTree.get(pageIndex);

Original file line number	Diff line number	Diff line change
`@@ -219,8 +219,9 @@ public int print(Graphics graphics, PageFormat pageFormat, int pageIndex)`
`219`	`219`	`float rasterDpi = dpi;`
`220`	`220`	`if (rasterDpi == RASTERIZE_DPI_AUTO)`
`221`	`221`	`{`
`222`		`- rasterDpi = new Matrix(graphics2D.getTransform()).getScalingFactorX() * 72.0f;`
`223`		`- LOG.debug("auto raster dpi: {}, g2d: {}, g2d transform: {}", rasterDpi, graphics2D, graphics2D.getTransform());`
	`222`	`+ AffineTransform transform = graphics2D.getTransform();`
	`223`	`+ rasterDpi = new Matrix(transform).getScalingFactorX() * 72.0f;`
	`224`	`+ LOG.debug("auto raster dpi: {}, g2d: {}, g2d transform: {}", rasterDpi, graphics2D, transform);`
`224`	`225`	`}`
`225`	`226`
`226`	`227`	`PDPage page = pageTree.get(pageIndex);`