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:39 UTC

svn commit: r1902602 - /pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/PDFSplit.java

Author: tilman
Date: Sat Jul  9 15:39:38 2022
New Revision: 1902602

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

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

Modified: pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/PDFSplit.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/PDFSplit.java?rev=1902602&r1=1902601&r2=1902602&view=diff
==============================================================================
--- pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/PDFSplit.java (original)
+++ pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/PDFSplit.java Sat Jul  9 15:39:38 2022
@@ -90,7 +90,6 @@ public final class PDFSplit implements C
 
         try (PDDocument document = Loader.loadPDF(infile, password))
         {
-            int numberOfPages = document.getNumberOfPages();
             boolean startEndPageSet = false;
             if (startPage != -1)
             {
@@ -98,6 +97,7 @@ public final class PDFSplit implements C
                 startEndPageSet = true;
                 if (split == -1)
                 {
+                    int numberOfPages = document.getNumberOfPages();
                     splitter.setSplitAtPage(numberOfPages);
                 }
             }