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 2015/01/14 23:18:22 UTC

svn commit: r1651854 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSFilterInputStream.java

Author: tilman
Date: Wed Jan 14 22:18:21 2015
New Revision: 1651854

URL: http://svn.apache.org/r1651854
Log:
PDFBOX-2576: No need to throw NullPointerException or IndexOutOfBoundsException, these will happen anyway

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSFilterInputStream.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSFilterInputStream.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSFilterInputStream.java?rev=1651854&r1=1651853&r2=1651854&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSFilterInputStream.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSFilterInputStream.java Wed Jan 14 22:18:21 2015
@@ -59,11 +59,8 @@ public class COSFilterInputStream extend
   @Override
   public int read(byte[] b, int off, int len) throws IOException
   {
-    if (b == null) {
-      throw new NullPointerException();
-    } else if (off < 0 || len < 0 || len > b.length - off) {
-        throw new IndexOutOfBoundsException();
-    } else if (len == 0) {
+    if (len == 0)
+    {
         return 0;
     }