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 2015/01/23 22:34:42 UTC

svn commit: r1654395 - /pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/imageio/JPEGUtil.java

Author: tilman
Date: Fri Jan 23 21:34:42 2015
New Revision: 1654395

URL: http://svn.apache.org/r1654395
Log:
PDFBOX-2576: make utility class final with private constructor; remove unused imports

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

Modified: pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/imageio/JPEGUtil.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/imageio/JPEGUtil.java?rev=1654395&r1=1654394&r2=1654395&view=diff
==============================================================================
--- pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/imageio/JPEGUtil.java (original)
+++ pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/imageio/JPEGUtil.java Fri Jan 23 21:34:42 2015
@@ -18,8 +18,6 @@ package org.apache.pdfbox.tools.imageio;
 import javax.imageio.metadata.IIOInvalidTreeException;
 import javax.imageio.metadata.IIOMetadata;
 import javax.imageio.metadata.IIOMetadataNode;
-import static org.apache.pdfbox.tools.imageio.MetaUtil.JPEG_NATIVE_FORMAT;
-import static org.apache.pdfbox.tools.imageio.MetaUtil.debugLogMetadata;
 import org.w3c.dom.Element;
 import org.w3c.dom.NodeList;
 
@@ -27,8 +25,12 @@ import org.w3c.dom.NodeList;
  *
  * @author Tilman Hausherr
  */
-class JPEGUtil
+final class JPEGUtil
 {
+    private JPEGUtil()
+    {
+    }
+    
     /**
      * Set dpi in a JPEG file
      *
@@ -86,6 +88,8 @@ class JPEGUtil
         {
             jfifChild.setAttribute("thumbHeight", "0");
         }
-        metadata.setFromTree(MetaUtil.JPEG_NATIVE_FORMAT, root); // mergeTree doesn't work for ARGB
+        
+        // mergeTree doesn't work for ARGB
+        metadata.setFromTree(MetaUtil.JPEG_NATIVE_FORMAT, root); 
     }
 }