InflaterInputStream (read): Don't return -1 unless the infate() call didn't deliver any output.

	* java/util/zip/InflaterInputStream (read): Don't return -1 unless
	the infate() call didn't deliver any output. Throw a ZipException if
	the needsDictionary() call returns true.
	* java/io/ByteArrayInputStream (read): Remove redundant bounds checks.
	* java/io/InputStreamReader: Use the default buffer size for the
	contained BufferedInputStream.

From-SVN: r37846

4 files changed, 18 insertions, 15 deletions

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index 5e6212f..57c04e0 100644
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -5,6 +5,13 @@
 	* configure.in: Check for setlocale.
 	* configure: Rebuilt.
 	* include/config.h.in: Rebuilt.
+	
+	* java/util/zip/InflaterInputStream (read): Don't return -1 unless
+	the infate() call didn't deliver any output. Throw a ZipException if
+	the needsDictionary() call returns true.
+	* java/io/ByteArrayInputStream (read): Remove redundant bounds checks.
+	* java/io/InputStreamReader: Use the default buffer size for the
+	contained BufferedInputStream.
 
 2000-11-28  Warren Levy  <warrenl@cygnus.com>
 
diff --git a/libjava/java/io/ByteArrayInputStream.java b/libjava/java/io/ByteArrayInputStream.java
index 97ec6e7..30ba8d7 100644
--- a/libjava/java/io/ByteArrayInputStream.java
+++ b/libjava/java/io/ByteArrayInputStream.java
@@ -72,9 +72,6 @@ public class ByteArrayInputStream extends InputStream
 
   public synchronized int read()
   {
-    if (pos < 0)
-      throw new ArrayIndexOutOfBoundsException(pos);
-
     if (pos < count)
       return ((int) buf[pos++]) & 0xFF;
     return -1;
@@ -82,10 +79,6 @@ public class ByteArrayInputStream extends InputStream
 
   public synchronized int read(byte[] b, int off, int len)
   {
-    /* Don't need to check pos value, arraycopy will check it. */
-    if (off < 0 || len < 0 || off + len > b.length)
-      throw new ArrayIndexOutOfBoundsException();
-
     if (pos >= count)
       return -1;
 
diff --git a/libjava/java/io/InputStreamReader.java b/libjava/java/io/InputStreamReader.java
index 478d8ef..73876fb 100644
--- a/libjava/java/io/InputStreamReader.java
+++ b/libjava/java/io/InputStreamReader.java
@@ -46,7 +46,7 @@ public class InputStreamReader extends Reader
   {
     this.in = in instanceof BufferedInputStream
               ? (BufferedInputStream) in
-              : new BufferedInputStream(in, 250);
+              : new BufferedInputStream(in);
     /* Don't need to call super(in) here as long as the lock gets set. */
     this.lock = in;
     converter = decoder;
diff --git a/libjava/java/util/zip/InflaterInputStream.java b/libjava/java/util/zip/InflaterInputStream.java
index 005c821..d7459eb 100644
--- a/libjava/java/util/zip/InflaterInputStream.java
+++ b/libjava/java/util/zip/InflaterInputStream.java
@@ -44,8 +44,6 @@ public class InflaterInputStream extends FilterInputStream
 {
   protected void fill () throws IOException
   {
-    if (inf == null)
-      throw new IOException ("stream closed");
     len = in.read(buf, 0, buf.length);
     if (len != -1)
       inf.setInput(buf, 0, len);
@@ -85,18 +83,23 @@ public class InflaterInputStream extends FilterInputStream
       return -1;
     if (inf.needsInput())
       fill ();
-    if (this.len == -1)
-      return -1; // Couldn't get any more data to feed to the Inflater
-    if (inf.needsDictionary())
-      return -1;
+    int count;
     try
       {
-	return inf.inflate(buf, off, len);
+	count = inf.inflate(buf, off, len);	
+	if (count == 0)
+	  {
+	    if (len == -1)
+	      return -1; // Couldn't get any more data to feed to the Inflater
+	    if (inf.needsDictionary())
+	      throw new ZipException ("Inflater needs Dictionary");
+	  }	      
       }
     catch (DataFormatException dfe)
       {
 	throw new ZipException (dfe.getMessage());
       }
+    return count;
   }
 
   public void close () throws IOException