You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2019/04/01 14:55:16 UTC

svn commit: r1856737 - /pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Splitter.java

Author: lehmi
Date: Mon Apr  1 14:55:16 2019
New Revision: 1856737

URL: http://svn.apache.org/viewvc?rev=1856737&view=rev
Log:
PDFBOX-4502: don't call getPage if out of range as proposed by ccouturi

Modified:
    pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Splitter.java

Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Splitter.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Splitter.java?rev=1856737&r1=1856736&r2=1856737&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Splitter.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Splitter.java Mon Apr  1 14:55:16 2019
@@ -143,10 +143,9 @@ public class Splitter
     {
         for (int i = 0; i < sourceDocument.getNumberOfPages(); i++)
         {
-            PDPage page = sourceDocument.getPage(i);
             if (currentPageNumber + 1 >= startPage && currentPageNumber + 1 <= endPage)
             {
-                processPage(page);
+                processPage(sourceDocument.getPage(i));
                 currentPageNumber++;
             }
             else