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 2019/07/30 19:23:37 UTC
svn commit: r1864020 -
/pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/imageio/ImageIOUtil.java
Author: tilman
Date: Tue Jul 30 19:23:37 2019
New Revision: 1864020
URL: http://svn.apache.org/viewvc?rev=1864020&view=rev
Log:
PDFBOX-4071: SonarQube fix
Modified:
pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/imageio/ImageIOUtil.java
Modified: pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/imageio/ImageIOUtil.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/imageio/ImageIOUtil.java?rev=1864020&r1=1864019&r2=1864020&view=diff
==============================================================================
--- pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/imageio/ImageIOUtil.java (original)
+++ pdfbox/branches/2.0/tools/src/main/java/org/apache/pdfbox/tools/imageio/ImageIOUtil.java Tue Jul 30 19:23:37 2019
@@ -289,8 +289,8 @@ public final class ImageIOUtil
// TIFF metadata
TIFFUtil.updateMetadata(metadata, image, dpi);
}
- else if ("jpeg".equals(formatName.toLowerCase())
- || "jpg".equals(formatName.toLowerCase()))
+ else if ("jpeg".equalsIgnoreCase(formatName)
+ || "jpg".equalsIgnoreCase(formatName))
{
// This segment must be run before other meta operations,
// or else "IIOInvalidTreeException: Invalid node: app0JFIF"
@@ -359,7 +359,7 @@ public final class ImageIOUtil
// PNG writer doesn't conform to the spec which is
// "The width of a pixel, in millimeters"
// but instead counts the pixels per millimeter
- float res = "PNG".equals(formatName.toUpperCase())
+ float res = "PNG".equalsIgnoreCase(formatName)
? dpi / 25.4f
: 25.4f / dpi;