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 2014/12/18 23:27:21 UTC
svn commit: r1646557 -
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java
Author: tilman
Date: Thu Dec 18 22:27:21 2014
New Revision: 1646557
URL: http://svn.apache.org/r1646557
Log:
PDFBOX-2576: code cleanup
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java?rev=1646557&r1=1646556&r2=1646557&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java Thu Dec 18 22:27:21 2014
@@ -2085,10 +2085,9 @@ public class NonSequentialPDFParser exte
*/
private boolean checkObjectId(String objectString, long offset) throws IOException
{
- boolean objectFound = false;
long originOffset = pdfSource.getOffset();
pdfSource.seek(offset);
- objectFound = checkBytesAtOffset(objectString.getBytes("ISO-8859-1"));
+ boolean objectFound = checkBytesAtOffset(objectString.getBytes("ISO-8859-1"));
pdfSource.seek(originOffset);
return objectFound;
}
@@ -2191,7 +2190,8 @@ public class NonSequentialPDFParser exte
}
}
currentOffset++;
- } while (!pdfSource.isEOF());
+ }
+ while (!pdfSource.isEOF());
// reestablish origin position
pdfSource.seek(originOffset);
}
@@ -2273,11 +2273,10 @@ public class NonSequentialPDFParser exte
// search backwards for the beginning of the stream
long newOffset = -1;
long xrefOffset = pdfSource.getOffset();
- long currentOffset = xrefOffset;
boolean objFound = false;
for (int i = 1; i < 30 && !objFound; i++)
{
- currentOffset = xrefOffset - (i * 10);
+ long currentOffset = xrefOffset - (i * 10);
if (currentOffset > 0)
{
pdfSource.seek(currentOffset);