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/11/13 18:17:25 UTC

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

Author: tilman
Date: Sat Nov 13 18:17:25 2021
New Revision: 1894997

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

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=1894997&r1=1894996&r2=1894997&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 Sat Nov 13 18:17:25 2021
@@ -266,7 +266,11 @@ public class Splitter
         createNewDocumentIfNecessary();
         
         PDPage imported = getDestinationDocument().importPage(page);
-        imported.setResources(page.getResources());
+        if (page.getResources() != null && !page.getCOSObject().containsKey(COSName.RESOURCES))
+        {
+            imported.setResources(page.getResources());
+            LOG.info("Resources imported in Splitter"); // follow-up to warning in importPage
+        }
         // remove page links to avoid copying not needed resources 
         processAnnotations(imported);
     }