You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by tb...@apache.org on 2012/11/18 15:24:20 UTC
svn commit: r1410892 -
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/PDFMarkedContentExtractor.java
Author: tboehme
Date: Sun Nov 18 14:24:19 2012
New Revision: 1410892
URL: http://svn.apache.org/viewvc?rev=1410892&view=rev
Log:
PDFBOX-1447: leave loop in PDFMarkedContentExtractor.processTextPosition() as soon as suppress character condition is set to'true'; removed unnecessary null check
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/PDFMarkedContentExtractor.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/PDFMarkedContentExtractor.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/PDFMarkedContentExtractor.java?rev=1410892&r1=1410891&r2=1410892&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/PDFMarkedContentExtractor.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/PDFMarkedContentExtractor.java Sun Nov 18 14:24:19 2012
@@ -190,7 +190,7 @@ public class PDFMarkedContentExtractor e
//
boolean suppressCharacter = false;
float tolerance = (text.getWidth()/textCharacter.length())/3.0f;
- for( int i=0; i<sameTextCharacters.size() && textCharacter != null; i++ )
+ for( int i=0; i<sameTextCharacters.size(); i++ )
{
TextPosition character = (TextPosition)sameTextCharacters.get( i );
String charCharacter = character.getCharacter();
@@ -206,6 +206,7 @@ public class PDFMarkedContentExtractor e
tolerance ) )
{
suppressCharacter = true;
+ break;
}
}
if( !suppressCharacter )