* java/util/ArrayList.jva (removeRange): If toIndex == fromIndex do

        nothing, if toIndex < fromIndex throw IndexOutIfBoundsException.


git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@51947 138bc75d-0d04-0410-961f-82ee72b054a4

2 files changed, 12 insertions, 3 deletions

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index 3c356d44800..a48ba515750 100644
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -1,3 +1,8 @@
+2002-04-05  Mark Wielaard <mark@klomp.org>
+
+	* java/util/ArrayList.jva (removeRange): If toIndex == fromIndex do
+	nothing, if toIndex < fromIndex throw IndexOutIfBoundsException.
+
 2002-04-05  Adam Megacz <adam@xwt.org>
 
         * exception.cc (abort): added static modifier
diff --git a/libjava/java/util/ArrayList.java b/libjava/java/util/ArrayList.java
index 34dc48ac179..59ce974b18f 100644
--- a/libjava/java/util/ArrayList.java
+++ b/libjava/java/util/ArrayList.java
@@ -437,19 +437,23 @@ public class ArrayList extends AbstractList
 
   /**
    * Removes all elements in the half-open interval [fromIndex, toIndex).
-   * You asked for it if you call this with invalid arguments.
+   * Does nothing when toIndex is equal to fromIndex.
    *
    * @param fromIndex the first index which will be removed
    * @param toIndex one greater than the last index which will be removed
+   * @throws IndexOutOfBoundsException if fromIndex &gt; toIndex
    */
   protected void removeRange(int fromIndex, int toIndex)
   {
-    if (fromIndex != toIndex)
+    int change = toIndex - fromIndex;
+    if (change > 0)
       {
         modCount++;
         System.arraycopy(data, toIndex, data, fromIndex, size - toIndex);
-        size -= toIndex - fromIndex;
+        size -= change;
       }
+    else if (change < 0)
+      throw new IndexOutOfBoundsException();
   }
 
   /**