diff options
author | Mario Torre <neugens@limasoftware.net> | 2008-03-26 11:23:40 +0000 |
---|---|---|
committer | Mario Torre <neugens@limasoftware.net> | 2008-03-26 11:23:40 +0000 |
commit | 5a8c7529d041ac57e035826759daf7fbdff437ee (patch) | |
tree | d5fe7de1b4c30081b878c9a0390047278abca412 /java | |
parent | 4258a7c70ddf43196e20cc016d5dd6072454b312 (diff) | |
download | classpath-5a8c7529d041ac57e035826759daf7fbdff437ee.tar.gz |
2008-03-26 Mario Torre <neugens@aicas.com>
* java/util/concurrent/CopyOnWriteArrayList.java (equals): removed
useless local variable.
(hashCode): new method.
Diffstat (limited to 'java')
-rw-r--r-- | java/util/concurrent/CopyOnWriteArrayList.java | 19 |
1 files changed, 14 insertions, 5 deletions
diff --git a/java/util/concurrent/CopyOnWriteArrayList.java b/java/util/concurrent/CopyOnWriteArrayList.java index aec1f8dc8..ac3b95be7 100644 --- a/java/util/concurrent/CopyOnWriteArrayList.java +++ b/java/util/concurrent/CopyOnWriteArrayList.java @@ -705,8 +705,6 @@ public class CopyOnWriteArrayList<E> if (this == o) return true; - boolean _equals = false; - // let's see if 'o' is a list, if so, we need to compare the elements // as returned by the iterator if (o instanceof List) @@ -723,10 +721,21 @@ public class CopyOnWriteArrayList<E> return false; } - _equals = true; + return true; } - - return _equals; + + return false; + } + + public int hashCode() + { + // see http://java.sun.com/6/docs/api/java/util/List.html#hashcode() + int hashcode = 1; + for (E element : this) + { + hashcode = 31 * hashcode + (element == null ? 0 : element.hashCode()); + } + return hashcode; } /** |