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

[tilman]

Author: tilman
Date: Sat May 18 16:21:29 2024
New Revision: 1917812

URL: http://svn.apache.org/viewvc?rev=1917812&view=rev
Log:
PDFBOX-5822: restore exception suggested by mkl in SO 78307200

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=1917812&r1=1917811&r2=1917812&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 
Sat May 18 16:21:29 2024
@@ -946,9 +946,15 @@ public class PDFTextStripper extends Leg
      * This will set the first page to be extracted by this class.
      *
      * @param startPageValue New value of 1-based startPage property.
+     * 
+     * @throws IllegalArgumentException if the parameter is below 1.
      */
     public void setStartPage(int startPageValue)
     {
+        if (startPageValue <= 0)
+        {
+            throw new IllegalArgumentException("Parameter must be 1-based, but 
is " + startPageValue);
+        }
         startPage = startPageValue;
     }
 
@@ -968,9 +974,15 @@ public class PDFTextStripper extends Leg
      * This will set the last page to be extracted by this class.
      *
      * @param endPageValue New value of 1-based endPage property.
+     *
+     * @throws IllegalArgumentException if the parameter is below 1.
      */
     public void setEndPage(int endPageValue)
     {
+        if (endPageValue <= 0)
+        {
+            throw new IllegalArgumentException("Parameter must be 1-based, but 
is " + endPageValue);
+        }
         endPage = endPageValue;
     }