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/06/12 15:12:59 UTC
svn commit: r1890731 -
/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Overlay.java
Author: tilman
Date: Sat Jun 12 15:12:59 2021
New Revision: 1890731
URL: http://svn.apache.org/viewvc?rev=1890731&view=rev
Log:
PDFBOX-4892: optimize, as suggested by valerybokov; add debug log
Modified:
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Overlay.java
Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Overlay.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Overlay.java?rev=1890731&r1=1890730&r2=1890731&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Overlay.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Overlay.java Sat Jun 12 15:12:59 2021
@@ -300,11 +300,10 @@ public class Overlay implements Closeabl
private Map<Integer,LayoutPage> getLayoutPages(PDDocument doc) throws IOException
{
- int numberOfPages = doc.getNumberOfPages();
- Map<Integer,LayoutPage> layoutPages = new HashMap<Integer, Overlay.LayoutPage>(numberOfPages);
- for (int i=0;i<numberOfPages;i++)
+ int i = 0;
+ Map<Integer, LayoutPage> layoutPages = new HashMap<Integer, LayoutPage>();
+ for (PDPage page : doc.getPages())
{
- PDPage page = doc.getPage(i);
COSBase contents = page.getCOSObject().getDictionaryObject(COSName.CONTENTS);
PDResources resources = page.getResources();
if (resources == null)
@@ -313,6 +312,7 @@ public class Overlay implements Closeabl
}
layoutPages.put(i, new LayoutPage(page.getMediaBox(), createCombinedContentStream(contents),
resources.getCOSObject(), page.getRotation()));
+ i++;
}
return layoutPages;
}