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 )