diff options
author | aph <aph@138bc75d-0d04-0410-961f-82ee72b054a4> | 2010-04-27 17:28:08 +0000 |
---|---|---|
committer | aph <aph@138bc75d-0d04-0410-961f-82ee72b054a4> | 2010-04-27 17:28:08 +0000 |
commit | a5090a34f55b9c13775844b9b70e8bc41e2aab03 (patch) | |
tree | 80beb6a6626fd922b0c49b8d3ddd5f5734887d92 /libjava/classpath/java/util/concurrent/CopyOnWriteArrayList.java | |
parent | 6861ea448041942cb26ca22854d1d34907682a4f (diff) | |
download | ppe42-gcc-a5090a34f55b9c13775844b9b70e8bc41e2aab03.tar.gz ppe42-gcc-a5090a34f55b9c13775844b9b70e8bc41e2aab03.zip |
2010-04-27 Andrew Haley <aph@redhat.com>
* java/util/concurrent/CopyOnWriteArrayList.java: Fix for empty
list.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@158790 138bc75d-0d04-0410-961f-82ee72b054a4
Diffstat (limited to 'libjava/classpath/java/util/concurrent/CopyOnWriteArrayList.java')
-rw-r--r-- | libjava/classpath/java/util/concurrent/CopyOnWriteArrayList.java | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/libjava/classpath/java/util/concurrent/CopyOnWriteArrayList.java b/libjava/classpath/java/util/concurrent/CopyOnWriteArrayList.java index 6e4fb9a8ac9..02071829269 100644 --- a/libjava/classpath/java/util/concurrent/CopyOnWriteArrayList.java +++ b/libjava/classpath/java/util/concurrent/CopyOnWriteArrayList.java @@ -452,7 +452,12 @@ public class CopyOnWriteArrayList<E> public synchronized boolean remove(Object element) { E[] snapshot = this.data; - E[] newData = (E[]) new Object[snapshot.length - 1]; + int len = snapshot.length; + + if (len == 0) + return false; + + E[] newData = (E[]) new Object[len - 1]; // search the element to remove while filling the backup array // this way we can run this method in O(n) |