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 2024/04/19 07:43:50 UTC

svn commit: r1917127 - /pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java

Author: tilman
Date: Fri Apr 19 07:43:50 2024
New Revision: 1917127

URL: http://svn.apache.org/viewvc?rev=1917127&view=rev
Log:
PDFBOX-5660: add warning / exception, as suggested by mkl in SO 78307200

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

Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java?rev=1917127&r1=1917126&r2=1917127&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java Fri Apr 19 07:43:50 2024
@@ -961,6 +961,10 @@ public class PDFTextStripper extends Leg
      */
     public void setStartPage(int startPageValue)
     {
+        if (startPageValue <= 0)
+        {
+            LOG.warn("Parameter must be 1-based, but is " + startPageValue);
+        }
         startPage = startPageValue;
     }
 
@@ -983,6 +987,10 @@ public class PDFTextStripper extends Leg
      */
     public void setEndPage(int endPageValue)
     {
+        if (endPageValue <= 0)
+        {
+            LOG.warn("Parameter must be 1-based, but is " + endPageValue);
+        }
         endPage = endPageValue;
     }