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 2021/09/26 13:01:08 UTC
svn commit: r1893647 - in
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive:
annotation/layout/PlainTextFormatter.java form/PlainTextFormatter.java
Author: tilman
Date: Sun Sep 26 13:01:08 2021
New Revision: 1893647
URL: http://svn.apache.org/viewvc?rev=1893647&view=rev
Log:
PDFBOX-4892: optimize, as suggested by valerybokov
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/annotation/layout/PlainTextFormatter.java
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PlainTextFormatter.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/annotation/layout/PlainTextFormatter.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/annotation/layout/PlainTextFormatter.java?rev=1893647&r1=1893646&r2=1893647&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/annotation/layout/PlainTextFormatter.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/annotation/layout/PlainTextFormatter.java Sun Sep 26 13:01:08 2021
@@ -271,15 +271,17 @@ public class PlainTextFormatter
lastPos += offset;
List<Word> words = line.getWords();
+ int wordIndex = 0;
for (Word word : words)
{
contents.showText(word.getText());
wordWidth = (Float) word.getAttributes().getIterator().getAttribute(TextAttribute.WIDTH);
- if (words.indexOf(word) != words.size() -1)
+ if (wordIndex != words.size() -1)
{
contents.newLineAtOffset(wordWidth + interWordSpacing, 0f);
lastPos = lastPos + wordWidth + interWordSpacing;
}
+ ++wordIndex;
}
}
horizontalOffset = horizontalOffset - lastPos;
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PlainTextFormatter.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PlainTextFormatter.java?rev=1893647&r1=1893646&r2=1893647&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PlainTextFormatter.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PlainTextFormatter.java Sun Sep 26 13:01:08 2021
@@ -271,15 +271,17 @@ class PlainTextFormatter
lastPos += offset;
List<Word> words = line.getWords();
+ int wordIndex = 0;
for (Word word : words)
{
contents.showText(word.getText());
wordWidth = (Float) word.getAttributes().getIterator().getAttribute(TextAttribute.WIDTH);
- if (words.indexOf(word) != words.size() -1)
+ if (wordIndex != words.size() -1)
{
contents.newLineAtOffset(wordWidth + interWordSpacing, 0f);
lastPos = lastPos + wordWidth + interWordSpacing;
}
+ ++wordIndex;
}
}
horizontalOffset = horizontalOffset - lastPos;