svn commit: r1936451 - pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/filter

[tilman]

Author: tilman
Date: Wed Jul 22 07:51:19 2026
New Revision: 1936451

Log:
PDFBOX-5660: improve log messages, as suggested by Valery Bokov; Sonar fix; 
closes #481

Modified:
   
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/filter/ASCIIHexFilter.java

Modified: 
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/filter/ASCIIHexFilter.java
==============================================================================
--- 
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/filter/ASCIIHexFilter.java
       Wed Jul 22 07:51:16 2026        (r1936450)
+++ 
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/filter/ASCIIHexFilter.java
       Wed Jul 22 07:51:19 2026        (r1936451)
@@ -67,7 +67,7 @@ final class ASCIIHexFilter extends Filte
     public DecodeResult decode(InputStream encoded, OutputStream decoded,
                                          COSDictionary parameters, int index) 
throws IOException
     {
-        int value, firstByte, secondByte;
+        int firstByte;
         while ((firstByte = encoded.read()) != -1)
         {
             // always after first char
@@ -82,10 +82,10 @@ final class ASCIIHexFilter extends Filte
        
             if (REVERSE_HEX[firstByte] == -1)
             {
-                LOG.error("Invalid hex, int: " + firstByte + " char: " + 
(char)firstByte);
+                LOG.error("Invalid hex, int: " + firstByte + " char: " + 
((char) firstByte) + " (1st byte)");
             }
-            value = REVERSE_HEX[firstByte] * 16;
-            secondByte = encoded.read();
+            int value = REVERSE_HEX[firstByte] * 16;
+            int secondByte = encoded.read();
        
             if (secondByte == -1 || isEOD(secondByte)) 
             {
@@ -95,7 +95,7 @@ final class ASCIIHexFilter extends Filte
             }
             if (REVERSE_HEX[secondByte] == -1)
             {
-                LOG.error("Invalid hex, int: " + secondByte + " char: " + 
(char) secondByte);
+                LOG.error("Invalid hex, int: " + secondByte + " char: " + 
((char) secondByte) + " (2nd byte)");
             }
             value += REVERSE_HEX[secondByte];
             decoded.write(value);