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 2022/07/09 15:39:35 UTC

svn commit: r1902601 - /pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/PDFSplit.java

Author: tilman
Date: Sat Jul  9 15:39:35 2022
New Revision: 1902601

URL: http://svn.apache.org/viewvc?rev=1902601&view=rev
Log:
PDFBOX-4892: optimize, as suggested by Valery Bokov

Modified:
    pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/PDFSplit.java

Modified: pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/PDFSplit.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/PDFSplit.java?rev=1902601&r1=1902600&r2=1902601&view=diff
==============================================================================
--- pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/PDFSplit.java (original)
+++ pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/PDFSplit.java Sat Jul  9 15:39:35 2022
@@ -139,7 +139,6 @@ public final class PDFSplit
             {
                 document = PDDocument.load(new File(pdfFile), password);
 
-                int numberOfPages = document.getNumberOfPages();
                 boolean startEndPageSet = false;
                 if (startPage != null)
                 {
@@ -147,6 +146,7 @@ public final class PDFSplit
                     startEndPageSet = true;
                     if (split == null)
                     {
+                        int numberOfPages = document.getNumberOfPages();
                         splitter.setSplitAtPage(numberOfPages);
                     }
                 }