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/02/05 10:48:41 UTC

svn commit: r1897789 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java

Author: tilman
Date: Sat Feb  5 10:48:41 2022
New Revision: 1897789

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

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java?rev=1897789&r1=1897788&r2=1897789&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java Sat Feb  5 10:48:41 2022
@@ -97,12 +97,16 @@ public class AcroFormOrphanWidgetsProces
 
         acroForm.setFields(fields);
 
-        // ensure that PDVariableText fields have the necessary resources
-        PDResources resources = acroForm.getDefaultResources();
+        PDResources resources = null;
         for (PDField field : acroForm.getFieldTree())
         {
             if (field instanceof PDVariableText)
             {
+                if (resources == null)
+                {
+                    // ensure that PDVariableText fields have the neccessary resources
+                    resources = acroForm.getDefaultResources();
+                }
                 ensureFontResources(resources, (PDVariableText) field);
             }
         }