You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by gb...@apache.org on 2012/12/16 11:46:57 UTC
svn commit: r1422513 -
/pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/process/TrailerValidationProcess.java
Author: gbailleul
Date: Sun Dec 16 10:46:56 2012
New Revision: 1422513
URL: http://svn.apache.org/viewvc?rev=1422513&view=rev
Log:
PDFBOX-1457: wasted work in TrailerValidationProcess.compareIds()
Modified:
pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/process/TrailerValidationProcess.java
Modified: pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/process/TrailerValidationProcess.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/process/TrailerValidationProcess.java?rev=1422513&r1=1422512&r2=1422513&view=diff
==============================================================================
--- pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/process/TrailerValidationProcess.java (original)
+++ pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/process/TrailerValidationProcess.java Sun Dec 16 10:46:56 2012
@@ -200,8 +200,13 @@ public class TrailerValidationProcess ex
// strings
if (!oneIsEquals)
oneIsEquals = ((COSString) ol).getString().equals(((COSString) of).getString());
+ else
+ break;
}
isEqual = isEqual && oneIsEquals;
+ if (!isEqual) {
+ break;
+ }
}
return isEqual;
}