From 5b77e15500b7b19d35976603dd71e8b997b2d8ea Mon Sep 17 00:00:00 2001 From: Sven Gothel Date: Thu, 17 Oct 2013 16:18:53 +0200 Subject: ArrayHashSet: Use final local vars if applicable --- src/java/com/jogamp/common/util/ArrayHashSet.java | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'src/java/com') diff --git a/src/java/com/jogamp/common/util/ArrayHashSet.java b/src/java/com/jogamp/common/util/ArrayHashSet.java index 3289924..ed3590a 100644 --- a/src/java/com/jogamp/common/util/ArrayHashSet.java +++ b/src/java/com/jogamp/common/util/ArrayHashSet.java @@ -95,9 +95,9 @@ public class ArrayHashSet */ @Override public final Object clone() { - ArrayList clonedList = new ArrayList(data); + final ArrayList clonedList = new ArrayList(data); - ArrayHashSet newObj = new ArrayHashSet(); + final ArrayHashSet newObj = new ArrayHashSet(); newObj.addAll(clonedList); return newObj; @@ -131,7 +131,7 @@ public class ArrayHashSet */ @Override public final boolean add(E element) { - boolean exists = map.containsKey(element); + final boolean exists = map.containsKey(element); if(!exists) { if(null != map.put(element, element)) { throw new InternalError("Already existing, but checked before: "+element); @@ -343,7 +343,7 @@ public class ArrayHashSet */ @Override public final E set(int index, E element) { - E old = remove(index); + final E old = remove(index); if(null!=old) { add(index, element); } @@ -359,7 +359,7 @@ public class ArrayHashSet */ @Override public final E remove(int index) { - E o = get(index); + final E o = get(index); if( null!=o && remove(o) ) { return o; } @@ -428,13 +428,13 @@ public class ArrayHashSet * or add the given key and return it. */ public final E getOrAdd(E key) { - E identity = get(key); + final E identity = get(key); if(null == identity) { // object not contained yet, add it if(!this.add(key)) { throw new InternalError("Key not mapped, but contained in list: "+key); } - identity = key; + return key; } return identity; } -- cgit v1.2.3