svn commit: r1928783 - pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor

[tilman]

Author: tilman
Date: Sat Sep 27 14:36:27 2025
New Revision: 1928783

Log:
PDFBOX-5660: optimize debugging; closes #265

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

Modified: 
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java
==============================================================================
--- 
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java
      Sat Sep 27 14:36:23 2025        (r1928782)
+++ 
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java
      Sat Sep 27 14:36:27 2025        (r1928783)
@@ -171,7 +171,10 @@ public class AcroFormOrphanWidgetsProces
                     if (acroFormResources.getFont(fontName) == null)
                     {
                         acroFormResources.put(fontName, 
widgetResources.getFont(fontName));
-                        LOG.debug("added font resource to AcroForm from widget 
for font name " + fontName.getName());
+                        if (LOG.isDebugEnabled())
+                        {
+                            LOG.debug("added font resource to AcroForm from 
widget for font name " + fontName.getName());
+                        }
                     }
                 }
                 catch (IOException ioe)
@@ -179,7 +182,7 @@ public class AcroFormOrphanWidgetsProces
                     LOG.debug("unable to add font to AcroForm for font name " 
+ fontName.getName());
                 }
             }
-            else
+            else if (LOG.isDebugEnabled())
             {
                 LOG.debug("font resource for widget was a subsetted font - 
ignored: " + fontName.getName());
             }
@@ -235,16 +238,22 @@ public class AcroFormOrphanWidgetsProces
             {
                 if (defaultResources.getFont(fontName) == null)
                 {
-                    LOG.debug("trying to add missing font resource for field " 
+ field.getFullyQualifiedName());
+                    if (LOG.isDebugEnabled())
+                    {
+                        LOG.debug("trying to add missing font resource for 
field " + field.getFullyQualifiedName());
+                    }
                     FontMapper mapper = FontMappers.instance();
                     FontMapping<TrueTypeFont> fontMapping = 
mapper.getTrueTypeFont(fontName.getName() , null);
                     if (fontMapping != null)
                     {
                         PDType0Font pdFont = PDType0Font.load(document, 
fontMapping.getFont(), false);
-                        LOG.debug("looked up font for " + fontName.getName() + 
" - found " + fontMapping.getFont().getName());
+                        if (LOG.isDebugEnabled())
+                        {
+                            LOG.debug("looked up font for " + 
fontName.getName() + " - found " + fontMapping.getFont().getName());
+                        }
                         defaultResources.put(fontName, pdFont);
                     }
-                    else
+                    else if (LOG.isDebugEnabled())
                     {
                         LOG.debug("no suitable font found for field " + 
field.getFullyQualifiedName() + " for font name " + fontName.getName());
                     }