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/12/02 20:21:21 UTC

svn commit: r1930209 - pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/imageio

Author: tilman
Date: Tue Dec  2 20:21:21 2025
New Revision: 1930209

Log:
PDFBOX-5660: optimize, as suggested by Valery Bokov, closes #360

Modified:
   pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/imageio/JPEGUtil.java

Modified: pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/imageio/JPEGUtil.java
==============================================================================
--- pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/imageio/JPEGUtil.java	Tue Dec  2 20:17:17 2025	(r1930208)
+++ pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/imageio/JPEGUtil.java	Tue Dec  2 20:21:21 2025	(r1930209)
@@ -78,8 +78,9 @@ final class JPEGUtil
             jfifChild.setAttribute("minorVersion", "2");
         }
         jfifChild.setAttribute("resUnits", "1"); // inch
-        jfifChild.setAttribute("Xdensity", Integer.toString(dpi));
-        jfifChild.setAttribute("Ydensity", Integer.toString(dpi));
+        String dpiString = Integer.toString(dpi);
+        jfifChild.setAttribute("Xdensity", dpiString);
+        jfifChild.setAttribute("Ydensity", dpiString);
         if (jfifChild.getAttribute("thumbWidth").isEmpty())
         {
             jfifChild.setAttribute("thumbWidth", "0");