From 126bac7bd6c3d7672f2ed913e53fbba65698d58c Mon Sep 17 00:00:00 2001 From: Andrew Haley Date: Tue, 27 Apr 2010 17:28:08 +0000 Subject: CopyOnWriteArrayList.java: Fix for empty list. 2010-04-27 Andrew Haley * java/util/concurrent/CopyOnWriteArrayList.java: Fix for empty list. From-SVN: r158790 --- libjava/classpath/java/util/concurrent/CopyOnWriteArrayList.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'libjava/classpath/java') diff --git a/libjava/classpath/java/util/concurrent/CopyOnWriteArrayList.java b/libjava/classpath/java/util/concurrent/CopyOnWriteArrayList.java index 6e4fb9a..0207182 100644 --- a/libjava/classpath/java/util/concurrent/CopyOnWriteArrayList.java +++ b/libjava/classpath/java/util/concurrent/CopyOnWriteArrayList.java @@ -452,7 +452,12 @@ public class CopyOnWriteArrayList 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) -- cgit v1.1