svn commit: r1934693 - in pdfbox/trunk/pdfbox/src: main/java/org/apache/pdfbox/pdmodel test/java/org/apache/pdfbox/pdmodel test/java/org/apache/pdfbox/pdmodel/common

[ti...@apache.org]

Author: tilman
Date: Wed May 27 18:05:24 2026
New Revision: 1934693

Log:
PDFBOX-6206: remove unneeded throws

Modified:
   pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDPage.java
   pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/TestPDPageAnnotationsFiltering.java
   pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/common/COSArrayListTest.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDPage.java
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDPage.java	Wed May 27 16:50:58 2026	(r1934692)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDPage.java	Wed May 27 18:05:24 2026	(r1934693)
@@ -775,10 +775,8 @@ public class PDPage implements COSObject
      *
      * @return List of the PDAnnotation objects, never null. The returned list is backed by the
      * annotations COSArray, so any adding or deleting in this list will change the document too.
-     * 
-     * @throws IOException If there is an error while creating the annotation list.
      */
-    public List<PDAnnotation> getAnnotations() throws IOException
+    public List<PDAnnotation> getAnnotations()
     {
         return getAnnotations(annotation -> true);
     }

Modified: pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/TestPDPageAnnotationsFiltering.java
==============================================================================
--- pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/TestPDPageAnnotationsFiltering.java	Wed May 27 16:50:58 2026	(r1934692)
+++ pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/TestPDPageAnnotationsFiltering.java	Wed May 27 18:05:24 2026	(r1934693)
@@ -19,7 +19,6 @@ package org.apache.pdfbox.pdmodel;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
-import java.io.IOException;
 import java.util.List;
 
 import org.apache.pdfbox.cos.COSArray;
@@ -57,7 +56,7 @@ class TestPDPageAnnotationsFiltering
     }
 
     @Test
-    void validateNoFiltering() throws IOException
+    void validateNoFiltering()
     {
         List<PDAnnotation> annotations = page.getAnnotations();
         assertEquals(3, annotations.size());
@@ -67,14 +66,14 @@ class TestPDPageAnnotationsFiltering
     }
 
     @Test
-    void validateAllFiltered() throws IOException
+    void validateAllFiltered()
     {
         List<PDAnnotation> annotations = page.getAnnotations(annotation -> false);
         assertEquals(0, annotations.size());
     }
 
     @Test
-    void validateSelectedFew() throws IOException
+    void validateSelectedFew()
     {
         List<PDAnnotation> annotations = page.getAnnotations(annotation -> 
             (annotation instanceof PDAnnotationLink || annotation instanceof PDAnnotationSquare));

Modified: pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/common/COSArrayListTest.java
==============================================================================
--- pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/common/COSArrayListTest.java	Wed May 27 16:50:58 2026	(r1934692)
+++ pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/common/COSArrayListTest.java	Wed May 27 18:05:24 2026	(r1934693)
@@ -275,7 +275,7 @@ class COSArrayListTest
     }
 
     @Test
-    void removeFromFilteredListByIndex() throws IOException
+    void removeFromFilteredListByIndex()
     {
         // retrieve all annotations from page but the link annotation
         // which is 2nd in list - see above setup
@@ -288,7 +288,7 @@ class COSArrayListTest
     }
 
     @Test
-    void removeFromFilteredListByObject() throws IOException
+    void removeFromFilteredListByObject()
     {
         // retrieve all annotations from page but the link annotation
         // which is 2nd in list - see above setup