commit-classpath
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

FYI: CharArrayReader fix.


From: Guilhem Lavaux
Subject: FYI: CharArrayReader fix.
Date: Sat, 27 Dec 2003 14:38:51 +0100
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5) Gecko/20031007

Hi,

I commited this patch to merge kaffe and classpath CharArrayReader. This modification has been done because of OutOfBounds new mauve testlet.

Guilhem.
Index: ChangeLog
===================================================================
RCS file: /cvsroot/classpath/classpath/ChangeLog,v
retrieving revision 1.1722
diff -u -r1.1722 ChangeLog
--- ChangeLog   27 Dec 2003 13:12:34 -0000      1.1722
+++ ChangeLog   27 Dec 2003 13:37:38 -0000
@@ -1,6 +1,12 @@
 2003-12-27  Guilhem Lavaux <address@hidden>
 
-       * java/io/BufferedReader (readLine): Make readLine() really block
+       * java/io/CharArrayReader.java (read): It should throw
+       IndexOutOfBoundsException and not ArrayIndexOutOfBoundsException (see
+       mauve).
+       
+2003-12-27  Guilhem Lavaux <address@hidden>
+
+       * java/io/BufferedReader.java (readLine): Make readLine() really block
        until either EOF is reached or a true error happens.
 
 2003-12-27  Michael Koch  <address@hidden>
Index: java/io/CharArrayReader.java
===================================================================
RCS file: /cvsroot/classpath/classpath/java/io/CharArrayReader.java,v
retrieving revision 1.5
diff -u -r1.5 CharArrayReader.java
--- java/io/CharArrayReader.java        22 Jan 2002 22:26:59 -0000      1.5
+++ java/io/CharArrayReader.java        27 Dec 2003 13:37:38 -0000
@@ -228,7 +228,7 @@
 
        /* Don't need to check pos value, arraycopy will check it. */
        if (off < 0 || len < 0 || off + len > b.length)
-         throw new ArrayIndexOutOfBoundsException();
+         throw new IndexOutOfBoundsException();
 
        if (pos >= count)
          return -1;

reply via email to

[Prev in Thread] Current Thread [Next in Thread]