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 2025/04/07 08:17:03 UTC

svn commit: r1924889 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationStamp.java

Author: tilman
Date: Mon Apr  7 08:17:03 2025
New Revision: 1924889

URL: http://svn.apache.org/viewvc?rev=1924889&view=rev
Log:
PDFBOX-5660: Sonar fix

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationStamp.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationStamp.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationStamp.java?rev=1924889&r1=1924888&r2=1924889&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationStamp.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationStamp.java Mon Apr  7 08:17:03 2025
@@ -148,14 +148,14 @@ public class FDFAnnotationStamp extends
         NodeList nodeList = appearanceXML.getChildNodes();
         String parentAttrKey = appearanceXML.getAttribute("KEY");
         LOG.debug("Appearance Root - tag: {}, name: {}, key: {}, children: {}",
-                appearanceXML.getTagName(), appearanceXML.getNodeName(), parentAttrKey,
-                nodeList.getLength());
+                appearanceXML::getTagName, appearanceXML::getNodeName, () -> parentAttrKey,
+                nodeList::getLength);
 
         // Currently only handles Appearance dictionary (AP key on the root)
         if (!"AP".equals(appearanceXML.getAttribute("KEY")))
         {
-            LOG.warn("{} => Not handling element: {} with key: {}", parentAttrKey,
-                    appearanceXML.getTagName(), appearanceXML.getAttribute("KEY"));
+            LOG.warn("{} => Not handling element: {} with key: {}", () -> parentAttrKey,
+                    appearanceXML::getTagName, () -> appearanceXML.getAttribute("KEY"));
             return dictionary;
         }
         for (int i = 0; i < nodeList.getLength(); i++)
@@ -168,9 +168,9 @@ public class FDFAnnotationStamp extends
                 if ("STREAM".equalsIgnoreCase(childTagName))
                 {
                     LOG.debug("{} => Process {} item in the dictionary after processing the {}",
-                            parentAttrKey, child.getAttribute("KEY"), childTagName);
+                            () -> parentAttrKey, () -> child.getAttribute("KEY"), () -> childTagName);
                     dictionary.setItem(child.getAttribute("KEY"), parseStreamElement(child));
-                    LOG.debug("{} => Set {}", parentAttrKey, child.getAttribute("KEY"));
+                    LOG.debug("{} => Set {}", () -> parentAttrKey, () -> child.getAttribute("KEY"));
                 }
                 else
                 {