You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pdfbox.apache.org by le...@apache.org on 2011/01/25 08:17:36 UTC

svn commit: r1063158 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/PDFTextStripper.java

Author: lehmi
Date: Tue Jan 25 07:17:35 2011
New Revision: 1063158

URL: http://svn.apache.org/viewvc?rev=1063158&view=rev
Log:
PDFBOX-939: fixed the word spacing in RTL dominant texts as proposed by Pavel Pisarev

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/PDFTextStripper.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/PDFTextStripper.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/PDFTextStripper.java?rev=1063158&r1=1063157&r2=1063158&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/PDFTextStripper.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/PDFTextStripper.java Tue Jan 25 07:17:35 2011
@@ -1812,7 +1812,7 @@ public class PDFTextStripper extends PDF
         int numberOfStrings = line.size();
         if (isRtlDominant) {
             for(int i=numberOfStrings-1; i>=0; i--){
-                if (i > 1)
+                if (i < numberOfStrings-1)
                     writeWordSeparator();
                 writeString(line.get(i));
             }