You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2021/05/24 15:35:37 UTC

svn commit: r1890164 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/printing/PDFPrintable.java

Author: tilman
Date: Mon May 24 15:35:36 2021
New Revision: 1890164

URL: http://svn.apache.org/viewvc?rev=1890164&view=rev
Log:
PDFBOX-5200: cache pageTree, as suggested by valerybokov

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/printing/PDFPrintable.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/printing/PDFPrintable.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/printing/PDFPrintable.java?rev=1890164&r1=1890163&r2=1890164&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/printing/PDFPrintable.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/printing/PDFPrintable.java Mon May 24 15:35:36 2021
@@ -31,6 +31,7 @@ import java.awt.print.PrinterIOException
 import java.io.IOException;
 import org.apache.pdfbox.pdmodel.PDDocument;
 import org.apache.pdfbox.pdmodel.PDPage;
+import org.apache.pdfbox.pdmodel.PDPageTree;
 import org.apache.pdfbox.pdmodel.common.PDRectangle;
 import org.apache.pdfbox.rendering.PDFRenderer;
 import org.apache.pdfbox.rendering.RenderDestination;
@@ -42,7 +43,7 @@ import org.apache.pdfbox.rendering.Rende
  */
 public final class PDFPrintable implements Printable
 {
-    private final PDDocument document;
+    private final PDPageTree pageTree;
     private final PDFRenderer renderer;
     
     private final boolean showPageBorder;
@@ -129,7 +130,7 @@ public final class PDFPrintable implemen
     public PDFPrintable(PDDocument document, Scaling scaling, boolean showPageBorder, float dpi,
                         boolean center, PDFRenderer renderer)
     {
-        this.document = document;
+        this.pageTree = document.getPages();
         this.renderer = renderer;
         this.scaling = scaling;
         this.showPageBorder = showPageBorder;
@@ -191,7 +192,7 @@ public final class PDFPrintable implemen
     public int print(Graphics graphics, PageFormat pageFormat, int pageIndex)
             throws PrinterException
     {
-        if (pageIndex < 0 || pageIndex >= document.getNumberOfPages())
+        if (pageIndex < 0 || pageIndex >= pageTree.getCount())
         {
             return NO_SUCH_PAGE;
         }
@@ -199,7 +200,7 @@ public final class PDFPrintable implemen
         {
             Graphics2D graphics2D = (Graphics2D)graphics;
 
-            PDPage page = document.getPage(pageIndex);
+            PDPage page = pageTree.get(pageIndex);
             PDRectangle cropBox = getRotatedCropBox(page);
 
             // the imageable area is the area within the page margins