svn commit: r1607883 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFParser.java

[tilman]

Author: tilman
Date: Fri Jul  4 15:56:38 2014
New Revision: 1607883

URL: http://svn.apache.org/r1607883
Log:
PDFBOX-2186: allow tab in xref lines, minor cleanup

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

Modified: 
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFParser.java
URL: 
http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFParser.java?rev=1607883&r1=1607882&r2=1607883&view=diff
==============================================================================
--- 
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFParser.java 
(original)
+++ 
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/PDFParser.java 
Fri Jul  4 15:56:38 2014
@@ -542,8 +542,8 @@ public class PDFParser extends BaseParse
         else
         {
             long number = -1;
-            int genNum = -1;
-            String objectKey = null;
+            int genNum;
+            String objectKey;
             boolean missingObjectNumber = false;
             try
             {
@@ -762,7 +762,7 @@ public class PDFParser extends BaseParse
                 }
                 //Ignore table contents
                 String currentLine = readLine();
-                String[] splitString = currentLine.split(" ");
+                String[] splitString = currentLine.split("\\s");
                 if (splitString.length < 3)
                 {
                     LOG.warn("invalid xref line: " + currentLine);
@@ -900,9 +900,9 @@ public class PDFParser extends BaseParse
     private static class ConflictObj
     {
 
-        private long offset;
-        private COSObjectKey objectKey;
-        private COSObject object;
+        private final long offset;
+        private final COSObjectKey objectKey;
+        private final COSObject object;
 
         ConflictObj(long offsetValue, COSObjectKey key, COSObject pdfObject)
         {
@@ -933,8 +933,7 @@ public class PDFParser extends BaseParse
                 do
                 {
                     ConflictObj o = conflicts.next();
-                    Long offset = new Long(o.offset);
-                    if (tolerantConflicResolver(values, offset, 4))
+                    if (tolerantConflicResolver(values, o.offset, 4))
                     {
                         COSObject pdfObject = 
document.getObjectFromPool(o.objectKey);
                         if (pdfObject.getObjectNumber() != null 
@@ -944,7 +943,7 @@ public class PDFParser extends BaseParse
                         }
                         else
                         {
-                            LOG.debug("Conflict object ["+o.objectKey+"] at 
offset "+offset
+                            LOG.debug("Conflict object [" + o.objectKey + "] 
at offset " + o.offset
                                     +" found in the xref table, but the object 
numbers differ. Ignoring this object."
                                     + " The document is maybe malformed.");
                         }