svn commit: r1890498 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java

[tilman]

Author: tilman
Date: Sat Jun  5 11:47:48 2021
New Revision: 1890498

URL: http://svn.apache.org/viewvc?rev=1890498&view=rev
Log:
PDFBOX-4892: improve log / exception messages

Modified:
    
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java

Modified: 
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java
URL: 
http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java?rev=1890498&r1=1890497&r2=1890498&view=diff
==============================================================================
--- 
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java 
(original)
+++ 
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java 
Sat Jun  5 11:47:48 2021
@@ -846,13 +846,15 @@ public abstract class BaseParser
             // This is not suppose to happen, but we will allow for it
             // so we are more compatible with POS writers that don't
             // follow the spec
+            long startOffset = source.getPosition();
             String badString = readString();
             if (badString.isEmpty())
             {
                 int peek = source.peek();
                 // we can end up in an infinite loop otherwise
                 throw new IOException("Unknown dir object c='" + c + "' cInt=" 
+ (int) c + " peek='"
-                        + (char) peek + "' peekInt=" + peek + " at offset " + 
source.getPosition());
+                        + (char) peek + "' peekInt=" + peek + " at offset " + 
source.getPosition()
+                        + " (start offset: " + startOffset + ")");
             }
 
             // if it's an endstream/endobj, we want to put it back so the 
caller will see it
@@ -863,7 +865,7 @@ public abstract class BaseParser
             else
             {
                 LOG.warn("Skipped unexpected dir object = '" + badString + "' 
at offset "
-                        + source.getPosition());
+                        + source.getPosition() + " (start offset: " + 
startOffset + ")");
             }
         }
         return null;