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/03/21 09:40:41 UTC

svn commit: r1887899 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java

Author: tilman
Date: Sun Mar 21 09:40:41 2021
New Revision: 1887899

URL: http://svn.apache.org/viewvc?rev=1887899&view=rev
Log:
PDFBOX-4892: add comment suggested by valerybokov

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

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java?rev=1887899&r1=1887898&r2=1887899&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java Sun Mar 21 09:40:41 2021
@@ -348,6 +348,8 @@ public class PDFTextStripper extends Leg
                 {
                     if (numberOfArticleSections < originalSize)
                     {
+                        //TODO Looks like decrement (--i) needed because next value will be ignored.
+                        // This segment is never reached in tests?!
                         charactersByArticle.remove(i);
                     }
                     else