commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ggreg...@apache.org
Subject commons-collections git commit: Use final for params.
Date Wed, 11 Oct 2017 22:25:24 GMT
Repository: commons-collections
Updated Branches:
  refs/heads/master 8713af90d -> cefe846e3


Use final for params.

Project: http://git-wip-us.apache.org/repos/asf/commons-collections/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-collections/commit/cefe846e
Tree: http://git-wip-us.apache.org/repos/asf/commons-collections/tree/cefe846e
Diff: http://git-wip-us.apache.org/repos/asf/commons-collections/diff/cefe846e

Branch: refs/heads/master
Commit: cefe846e3e4a239dd568623346160ad41d7256ab
Parents: 8713af9
Author: Gary Gregory <ggregory@apache.org>
Authored: Wed Oct 11 16:25:21 2017 -0600
Committer: Gary Gregory <ggregory@apache.org>
Committed: Wed Oct 11 16:25:21 2017 -0600

----------------------------------------------------------------------
 .../commons/collections4/CollectionUtils.java   | 10 ++--
 .../commons/collections4/IterableUtils.java     |  4 +-
 .../commons/collections4/IteratorUtils.java     |  6 +--
 .../apache/commons/collections4/ListUtils.java  |  2 +-
 .../apache/commons/collections4/MapUtils.java   |  2 +-
 .../apache/commons/collections4/SetUtils.java   | 12 ++---
 .../collections4/keyvalue/AbstractKeyValue.java |  4 +-
 .../collections4/map/AbstractReferenceMap.java  |  2 +-
 .../commons/collections4/map/MultiValueMap.java |  6 +--
 .../multimap/AbstractListValuedMap.java         | 28 +++++------
 .../multimap/AbstractMultiValuedMap.java        | 52 ++++++++++----------
 .../AbstractMultiValuedMapDecorator.java        |  8 +--
 .../multimap/AbstractSetValuedMap.java          |  6 +--
 .../multimap/ArrayListValuedHashMap.java        |  8 +--
 .../multimap/HashSetValuedHashMap.java          |  8 +--
 .../multimap/UnmodifiableMultiValuedMap.java    |  2 +-
 .../multiset/AbstractMapMultiSet.java           |  2 +-
 .../collections4/multiset/AbstractMultiSet.java |  4 +-
 .../multiset/AbstractMultiSetDecorator.java     |  2 +-
 .../multiset/PredicatedMultiSet.java            |  2 +-
 .../multiset/SynchronizedMultiSet.java          |  2 +-
 .../multiset/UnmodifiableMultiSet.java          |  2 +-
 .../collections4/queue/CircularFifoQueue.java   |  2 +-
 .../set/AbstractNavigableSetDecorator.java      | 14 +++---
 .../set/PredicatedNavigableSet.java             | 14 +++---
 .../set/TransformedNavigableSet.java            | 14 +++---
 .../set/UnmodifiableNavigableSet.java           |  6 +--
 .../collections4/trie/UnmodifiableTrie.java     |  4 +-
 .../collections4/CollectionUtilsTest.java       | 16 +++---
 .../collections4/FluentIterableTest.java        |  6 +--
 .../commons/collections4/IterableUtilsTest.java | 24 ++++-----
 .../commons/collections4/MapUtilsTest.java      |  6 +--
 .../collections4/TransformerUtilsTest.java      |  2 +-
 .../commons/collections4/bag/TreeBagTest.java   |  2 +-
 .../PredicatedCollectionBuilderTest.java        |  2 +-
 .../iterators/PeekingIteratorTest.java          |  2 +-
 .../iterators/PushbackIteratorTest.java         |  2 +-
 .../collections4/keyvalue/MultiKeyTest.java     |  2 +-
 .../collections4/map/MultiValueMapTest.java     |  4 +-
 .../map/PassiveExpiringMapTest.java             |  2 +-
 .../multimap/AbstractMultiValuedMapTest.java    |  6 +--
 .../multimap/ArrayListValuedHashMapTest.java    |  2 +-
 .../multimap/HashSetValuedHashMapTest.java      |  2 +-
 .../multimap/TransformedMultiValuedMapTest.java |  2 +-
 .../UnmodifiableMultiValuedMapTest.java         |  2 +-
 45 files changed, 156 insertions(+), 156 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/CollectionUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/CollectionUtils.java b/src/main/java/org/apache/commons/collections4/CollectionUtils.java
index 5b974cb..1b54be5 100644
--- a/src/main/java/org/apache/commons/collections4/CollectionUtils.java
+++ b/src/main/java/org/apache/commons/collections4/CollectionUtils.java
@@ -923,7 +923,7 @@ public class CollectionUtils {
      * @since 4.1
      */
     public static <O, R extends Collection<? super O>> R select(final Iterable<? extends O> inputCollection,
-            final Predicate<? super O> predicate, R outputCollection, R rejectedCollection) {
+            final Predicate<? super O> predicate, final R outputCollection, final R rejectedCollection) {
 
         if (inputCollection != null && predicate != null) {
             for (final O element : inputCollection) {
@@ -1506,8 +1506,8 @@ public class CollectionUtils {
      * @throws NullPointerException if either collection is null
      * @since 4.0
      */
-    public static <O extends Comparable<? super O>> List<O> collate(Iterable<? extends O> a,
-                                                                    Iterable<? extends O> b) {
+    public static <O extends Comparable<? super O>> List<O> collate(final Iterable<? extends O> a,
+                                                                    final Iterable<? extends O> b) {
         return collate(a, b, ComparatorUtils.<O>naturalComparator(), true);
     }
 
@@ -1688,7 +1688,7 @@ public class CollectionUtils {
 
         final Transformer<E, EquatorWrapper<E>> transformer = new Transformer<E, EquatorWrapper<E>>() {
             @Override
-            public EquatorWrapper<E> transform(E input) {
+            public EquatorWrapper<E> transform(final E input) {
                 return new EquatorWrapper<>(equator, input);
             }
         };
@@ -1764,7 +1764,7 @@ public class CollectionUtils {
 
         final Transformer<E, EquatorWrapper<E>> transformer = new Transformer<E, EquatorWrapper<E>>() {
             @Override
-            public EquatorWrapper<E> transform(E input) {
+            public EquatorWrapper<E> transform(final E input) {
                 return new EquatorWrapper<>(equator, input);
             }
         };

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/IterableUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/IterableUtils.java b/src/main/java/org/apache/commons/collections4/IterableUtils.java
index ad24eeb..15e50c2 100644
--- a/src/main/java/org/apache/commons/collections4/IterableUtils.java
+++ b/src/main/java/org/apache/commons/collections4/IterableUtils.java
@@ -172,7 +172,7 @@ public class IterableUtils {
             public Iterator<E> iterator() {
                 return new LazyIteratorChain<E>() {
                     @Override
-                    protected Iterator<? extends E> nextIterator(int count) {
+                    protected Iterator<? extends E> nextIterator(final int count) {
                         if (count > iterables.length) {
                             return null;
                         } else {
@@ -325,7 +325,7 @@ public class IterableUtils {
             public Iterator<E> iterator() {
                 return new LazyIteratorChain<E>() {
                     @Override
-                    protected Iterator<? extends E> nextIterator(int count) {
+                    protected Iterator<? extends E> nextIterator(final int count) {
                         if (IterableUtils.isEmpty(iterable)) {
                             return null;
                         } else {

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/IteratorUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/IteratorUtils.java b/src/main/java/org/apache/commons/collections4/IteratorUtils.java
index cfce5b3..e4b30d3 100644
--- a/src/main/java/org/apache/commons/collections4/IteratorUtils.java
+++ b/src/main/java/org/apache/commons/collections4/IteratorUtils.java
@@ -433,7 +433,7 @@ public class IteratorUtils {
      * @throws IllegalArgumentException if max is negative
      * @since 4.1
      */
-    public static <E> BoundedIterator<E> boundedIterator(final Iterator<? extends E> iterator, long max) {
+    public static <E> BoundedIterator<E> boundedIterator(final Iterator<? extends E> iterator, final long max) {
         return boundedIterator(iterator, 0, max);
     }
 
@@ -455,7 +455,7 @@ public class IteratorUtils {
      * @since 4.1
      */
     public static <E> BoundedIterator<E> boundedIterator(final Iterator<? extends E> iterator,
-                                                         long offset, long max) {
+                                                         final long offset, final long max) {
         return new BoundedIterator<>(iterator, offset, max);
     }
 
@@ -890,7 +890,7 @@ public class IteratorUtils {
      * @throws IllegalArgumentException if offset is negative
      * @since 4.1
      */
-    public static <E> SkippingIterator<E> skippingIterator(final Iterator<E> iterator, long offset) {
+    public static <E> SkippingIterator<E> skippingIterator(final Iterator<E> iterator, final long offset) {
         return new SkippingIterator<>(iterator, offset);
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/ListUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/ListUtils.java b/src/main/java/org/apache/commons/collections4/ListUtils.java
index 38e2e8b..c5fe813 100644
--- a/src/main/java/org/apache/commons/collections4/ListUtils.java
+++ b/src/main/java/org/apache/commons/collections4/ListUtils.java
@@ -613,7 +613,7 @@ public class ListUtils {
       }
 
       @Override
-      public Character get( int index ) {
+      public Character get( final int index ) {
         return Character.valueOf(sequence.charAt( index ));
       }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/MapUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/MapUtils.java b/src/main/java/org/apache/commons/collections4/MapUtils.java
index 15dfe82..30a084e 100644
--- a/src/main/java/org/apache/commons/collections4/MapUtils.java
+++ b/src/main/java/org/apache/commons/collections4/MapUtils.java
@@ -1798,7 +1798,7 @@ public class MapUtils {
      * @param map a Map or null
      * @return the given map size or 0 if the map is null
      */
-    public static int size(Map<?, ?> map) {
+    public static int size(final Map<?, ?> map) {
         return map == null ? 0 : map.size();
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/SetUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/SetUtils.java b/src/main/java/org/apache/commons/collections4/SetUtils.java
index e97b9c6..3d24cb2 100644
--- a/src/main/java/org/apache/commons/collections4/SetUtils.java
+++ b/src/main/java/org/apache/commons/collections4/SetUtils.java
@@ -446,7 +446,7 @@ public class SetUtils {
 
         return new SetView<E>() {
             @Override
-            public boolean contains(Object o) {
+            public boolean contains(final Object o) {
                 return a.contains(o) || b.contains(o);
             }
 
@@ -488,14 +488,14 @@ public class SetUtils {
 
         final Predicate<E> notContainedInB = new Predicate<E>() {
             @Override
-            public boolean evaluate(E object) {
+            public boolean evaluate(final E object) {
                 return !b.contains(object);
             }
         };
 
         return new SetView<E>() {
             @Override
-            public boolean contains(Object o) {
+            public boolean contains(final Object o) {
                 return a.contains(o) && !b.contains(o);
             }
 
@@ -526,14 +526,14 @@ public class SetUtils {
 
         final Predicate<E> containedInB = new Predicate<E>() {
             @Override
-            public boolean evaluate(E object) {
+            public boolean evaluate(final E object) {
                 return b.contains(object);
             }
         };
 
         return new SetView<E>() {
             @Override
-            public boolean contains(Object o) {
+            public boolean contains(final Object o) {
                 return a.contains(o) && b.contains(o);
             }
 
@@ -570,7 +570,7 @@ public class SetUtils {
 
         return new SetView<E>() {
             @Override
-            public boolean contains(Object o) {
+            public boolean contains(final Object o) {
                 return a.contains(o) ^ b.contains(o);
             }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/keyvalue/AbstractKeyValue.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/keyvalue/AbstractKeyValue.java b/src/main/java/org/apache/commons/collections4/keyvalue/AbstractKeyValue.java
index f816561..6f7a5e1 100644
--- a/src/main/java/org/apache/commons/collections4/keyvalue/AbstractKeyValue.java
+++ b/src/main/java/org/apache/commons/collections4/keyvalue/AbstractKeyValue.java
@@ -53,7 +53,7 @@ public abstract class AbstractKeyValue<K, V> implements KeyValue<K, V> {
         return key;
     }
 
-    protected K setKey(K key) {
+    protected K setKey(final K key) {
         final K old = this.key;
         this.key = key;
         return old;
@@ -69,7 +69,7 @@ public abstract class AbstractKeyValue<K, V> implements KeyValue<K, V> {
         return value;
     }
 
-    protected V setValue(V value) {
+    protected V setValue(final V value) {
         final V old = this.value;
         this.value = value;
         return old;

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/map/AbstractReferenceMap.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/map/AbstractReferenceMap.java b/src/main/java/org/apache/commons/collections4/map/AbstractReferenceMap.java
index 82ac7e3..17fd4cb 100644
--- a/src/main/java/org/apache/commons/collections4/map/AbstractReferenceMap.java
+++ b/src/main/java/org/apache/commons/collections4/map/AbstractReferenceMap.java
@@ -1059,7 +1059,7 @@ public abstract class AbstractReferenceMap<K, V> extends AbstractHashedMap<K, V>
      * @param type the type to check against.
      * @return true if keyType has the specified type
      */
-    protected boolean isKeyType(ReferenceStrength type) {
+    protected boolean isKeyType(final ReferenceStrength type) {
         return this.keyType == type;
     }
 }

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/map/MultiValueMap.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/map/MultiValueMap.java b/src/main/java/org/apache/commons/collections4/map/MultiValueMap.java
index a929006..98e45da 100644
--- a/src/main/java/org/apache/commons/collections4/map/MultiValueMap.java
+++ b/src/main/java/org/apache/commons/collections4/map/MultiValueMap.java
@@ -427,7 +427,7 @@ public class MultiValueMap<K, V> extends AbstractMapDecorator<K, Object> impleme
 
         return new LazyIteratorChain<Entry<K, V>>() {
             @Override
-            protected Iterator<? extends Entry<K, V>> nextIterator(int count) {
+            protected Iterator<? extends Entry<K, V>> nextIterator(final int count) {
                 if ( ! keyIterator.hasNext() ) {
                     return null;
                 }
@@ -445,7 +445,7 @@ public class MultiValueMap<K, V> extends AbstractMapDecorator<K, Object> impleme
                                 return input;
                             }
                             @Override
-                            public V setValue(V value) {
+                            public V setValue(final V value) {
                                 throw new UnsupportedOperationException();
                             }
                         };
@@ -564,7 +564,7 @@ public class MultiValueMap<K, V> extends AbstractMapDecorator<K, Object> impleme
             }
         }
 
-        private void readObject(ObjectInputStream is) throws IOException, ClassNotFoundException {
+        private void readObject(final ObjectInputStream is) throws IOException, ClassNotFoundException {
             is.defaultReadObject();
             // ensure that the de-serialized class is a Collection, COLLECTIONS-580
             if (clazz != null && !Collection.class.isAssignableFrom(clazz)) {

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multimap/AbstractListValuedMap.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multimap/AbstractListValuedMap.java b/src/main/java/org/apache/commons/collections4/multimap/AbstractListValuedMap.java
index 8904f6c..ffe6978 100644
--- a/src/main/java/org/apache/commons/collections4/multimap/AbstractListValuedMap.java
+++ b/src/main/java/org/apache/commons/collections4/multimap/AbstractListValuedMap.java
@@ -96,7 +96,7 @@ public abstract class AbstractListValuedMap<K, V> extends AbstractMultiValuedMap
      *   unmodifiable list for no mapping found.
      */
     @Override
-    public List<V> remove(Object key) {
+    public List<V> remove(final Object key) {
         return ListUtils.emptyIfNull(getMap().remove(key));
     }
 
@@ -116,7 +116,7 @@ public abstract class AbstractListValuedMap<K, V> extends AbstractMultiValuedMap
         }
 
         @Override
-        public void add(int index, V value) {
+        public void add(final int index, final V value) {
             List<V> list = getMapping();
             if (list == null) {
                 list = createCollection();
@@ -126,7 +126,7 @@ public abstract class AbstractListValuedMap<K, V> extends AbstractMultiValuedMap
         }
 
         @Override
-        public boolean addAll(int index, Collection<? extends V> c) {
+        public boolean addAll(final int index, final Collection<? extends V> c) {
             List<V> list = getMapping();
             if (list == null) {
                 list = createCollection();
@@ -140,19 +140,19 @@ public abstract class AbstractListValuedMap<K, V> extends AbstractMultiValuedMap
         }
 
         @Override
-        public V get(int index) {
+        public V get(final int index) {
             final List<V> list = ListUtils.emptyIfNull(getMapping());
             return list.get(index);
         }
 
         @Override
-        public int indexOf(Object o) {
+        public int indexOf(final Object o) {
             final List<V> list = ListUtils.emptyIfNull(getMapping());
             return list.indexOf(o);
         }
 
         @Override
-        public int lastIndexOf(Object o) {
+        public int lastIndexOf(final Object o) {
             final List<V> list = ListUtils.emptyIfNull(getMapping());
             return list.lastIndexOf(o);
         }
@@ -163,12 +163,12 @@ public abstract class AbstractListValuedMap<K, V> extends AbstractMultiValuedMap
         }
 
         @Override
-        public ListIterator<V> listIterator(int index) {
+        public ListIterator<V> listIterator(final int index) {
             return new ValuesListIterator(key, index);
         }
 
         @Override
-        public V remove(int index) {
+        public V remove(final int index) {
             final List<V> list = ListUtils.emptyIfNull(getMapping());
             V value = list.remove(index);
             if (list.isEmpty()) {
@@ -178,19 +178,19 @@ public abstract class AbstractListValuedMap<K, V> extends AbstractMultiValuedMap
         }
 
         @Override
-        public V set(int index, V value) {
+        public V set(final int index, final V value) {
             final List<V> list = ListUtils.emptyIfNull(getMapping());
             return list.set(index, value);
         }
 
         @Override
-        public List<V> subList(int fromIndex, int toIndex) {
+        public List<V> subList(final int fromIndex, final int toIndex) {
             final List<V> list = ListUtils.emptyIfNull(getMapping());
             return list.subList(fromIndex, toIndex);
         }
 
         @Override
-        public boolean equals(Object other) {
+        public boolean equals(final Object other) {
             final List<V> list = getMapping();
             if (list == null) {
                 return Collections.emptyList().equals(other);
@@ -223,14 +223,14 @@ public abstract class AbstractListValuedMap<K, V> extends AbstractMultiValuedMap
             this.iterator = values.listIterator();
         }
 
-        public ValuesListIterator(final K key, int index) {
+        public ValuesListIterator(final K key, final int index) {
             this.key = key;
             this.values = ListUtils.emptyIfNull(getMap().get(key));
             this.iterator = values.listIterator(index);
         }
 
         @Override
-        public void add(V value) {
+        public void add(final V value) {
             if (getMap().get(key) == null) {
                 List<V> list = createCollection();
                 getMap().put(key, list);
@@ -279,7 +279,7 @@ public abstract class AbstractListValuedMap<K, V> extends AbstractMultiValuedMap
         }
 
         @Override
-        public void set(V value) {
+        public void set(final V value) {
             iterator.set(value);
         }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java b/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java
index 620667b..8927185 100644
--- a/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java
+++ b/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java
@@ -109,7 +109,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
      * @param map the map to wrap
      */
     @SuppressWarnings("unchecked")
-    protected void setMap(Map<K, ? extends Collection<V>> map) {
+    protected void setMap(final Map<K, ? extends Collection<V>> map) {
         this.map = (Map<K, Collection<V>>) map;
     }
 
@@ -117,7 +117,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
 
     // -----------------------------------------------------------------------
     @Override
-    public boolean containsKey(Object key) {
+    public boolean containsKey(final Object key) {
         return getMap().containsKey(key);
     }
 
@@ -127,7 +127,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
     }
 
     @Override
-    public boolean containsMapping(Object key, Object value) {
+    public boolean containsMapping(final Object key, final Object value) {
         Collection<V> coll = getMap().get(key);
         return coll != null && coll.contains(value);
     }
@@ -163,7 +163,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
      *   empty, unmodifiable collection for no mapping found
      */
     @Override
-    public Collection<V> remove(Object key) {
+    public Collection<V> remove(final Object key) {
         return CollectionUtils.emptyIfNull(getMap().remove(key));
     }
 
@@ -370,7 +370,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
     }
 
     @Override
-    public boolean equals(Object obj) {
+    public boolean equals(final Object obj) {
         if (this == obj) {
             return true;
         }
@@ -415,7 +415,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
         }
 
         @Override
-        public boolean add(V value) {
+        public boolean add(final V value) {
             Collection<V> coll = getMapping();
             if (coll == null) {
                 coll = createCollection();
@@ -425,7 +425,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
         }
 
         @Override
-        public boolean addAll(Collection<? extends V> other) {
+        public boolean addAll(final Collection<? extends V> other) {
             Collection<V> coll = getMapping();
             if (coll == null) {
                 coll = createCollection();
@@ -460,13 +460,13 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
         }
 
         @Override
-        public boolean contains(Object obj) {
+        public boolean contains(final Object obj) {
             final Collection<V> coll = getMapping();
             return coll == null ? false : coll.contains(obj);
         }
 
         @Override
-        public boolean containsAll(Collection<?> other) {
+        public boolean containsAll(final Collection<?> other) {
             final Collection<V> coll = getMapping();
             return coll == null ? false : coll.containsAll(other);
         }
@@ -478,7 +478,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
         }
 
         @Override
-        public boolean remove(Object item) {
+        public boolean remove(final Object item) {
             final Collection<V> coll = getMapping();
             if (coll == null) {
                 return false;
@@ -492,7 +492,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
         }
 
         @Override
-        public boolean removeAll(Collection<?> c) {
+        public boolean removeAll(final Collection<?> c) {
             final Collection<V> coll = getMapping();
             if (coll == null) {
                 return false;
@@ -506,7 +506,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
         }
 
         @Override
-        public boolean retainAll(Collection<?> c) {
+        public boolean retainAll(final Collection<?> c) {
             final Collection<V> coll = getMapping();
             if (coll == null) {
                 return false;
@@ -530,7 +530,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
 
         @Override
         @SuppressWarnings("unchecked")
-        public <T> T[] toArray(T[] a) {
+        public <T> T[] toArray(final T[] a) {
             final Collection<V> coll = getMapping();
             if (coll == null) {
                 return (T[]) CollectionUtils.EMPTY_COLLECTION.toArray(a);
@@ -555,7 +555,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
     private class KeysMultiSet extends AbstractMultiSet<K> {
 
         @Override
-        public boolean contains(Object o) {
+        public boolean contains(final Object o) {
             return getMap().containsKey(o);
         }
 
@@ -575,7 +575,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
         }
 
         @Override
-        public int getCount(Object object) {
+        public int getCount(final Object object) {
             int count = 0;
             Collection<V> col = AbstractMultiValuedMap.this.getMap().get(object);
             if (col != null) {
@@ -622,7 +622,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
                 final Iterator<K> keyIterator = keysCol.iterator();
 
                 @Override
-                protected Iterator<? extends Entry<K, V>> nextIterator(int count) {
+                protected Iterator<? extends Entry<K, V>> nextIterator(final int count) {
                     if (!keyIterator.hasNext()) {
                         return null;
                     }
@@ -652,12 +652,12 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
      */
     private class MultiValuedMapEntry extends AbstractMapEntry<K, V> {
 
-        public MultiValuedMapEntry(K key, V value) {
+        public MultiValuedMapEntry(final K key, final V value) {
             super(key, value);
         }
 
         @Override
-        public V setValue(V value) {
+        public V setValue(final V value) {
             throw new UnsupportedOperationException();
         }
 
@@ -709,7 +709,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
         }
 
         @Override
-        public V setValue(V value) {
+        public V setValue(final V value) {
             if (current == null) {
                 throw new IllegalStateException();
             }
@@ -791,12 +791,12 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
         }
 
         @Override
-        public boolean containsKey(Object key) {
+        public boolean containsKey(final Object key) {
             return decoratedMap.containsKey(key);
         }
 
         @Override
-        public Collection<V> get(Object key) {
+        public Collection<V> get(final Object key) {
           Collection<V> collection = decoratedMap.get(key);
           if (collection == null) {
             return null;
@@ -817,7 +817,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
         }
 
         @Override
-        public Collection<V> remove(Object key) {
+        public Collection<V> remove(final Object key) {
           Collection<V> collection = decoratedMap.remove(key);
           if (collection == null) {
             return null;
@@ -830,7 +830,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
         }
 
         @Override
-        public boolean equals(Object object) {
+        public boolean equals(final Object object) {
           return this == object || decoratedMap.equals(object);
         }
 
@@ -867,12 +867,12 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
             }
 
             @Override
-            public boolean contains(Object o) {
+            public boolean contains(final Object o) {
                 return decoratedMap.entrySet().contains(o);
             }
 
             @Override
-            public boolean remove(Object o) {
+            public boolean remove(final Object o) {
                 if (!contains(o)) {
                     return false;
                 }
@@ -887,7 +887,7 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K,
          */
         class AsMapEntrySetIterator extends AbstractIteratorDecorator<Map.Entry<K, Collection<V>>> {
 
-            AsMapEntrySetIterator(Iterator<Map.Entry<K, Collection<V>>> iterator) {
+            AsMapEntrySetIterator(final Iterator<Map.Entry<K, Collection<V>>> iterator) {
                 super(iterator);
             }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapDecorator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapDecorator.java b/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapDecorator.java
index d49e7a0..8a6ff63 100644
--- a/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapDecorator.java
+++ b/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapDecorator.java
@@ -117,7 +117,7 @@ public abstract class AbstractMultiValuedMapDecorator<K, V>
     }
 
     @Override
-    public boolean put(K key, V value) {
+    public boolean put(final K key, final V value) {
         return decorated().put(key, value);
     }
 
@@ -147,17 +147,17 @@ public abstract class AbstractMultiValuedMapDecorator<K, V>
     }
 
     @Override
-    public boolean putAll(K key, Iterable<? extends V> values) {
+    public boolean putAll(final K key, final Iterable<? extends V> values) {
         return decorated().putAll(key, values);
     }
 
     @Override
-    public boolean putAll(Map<? extends K, ? extends V> map) {
+    public boolean putAll(final Map<? extends K, ? extends V> map) {
         return decorated().putAll(map);
     }
 
     @Override
-    public boolean putAll(MultiValuedMap<? extends K, ? extends V> map) {
+    public boolean putAll(final MultiValuedMap<? extends K, ? extends V> map) {
         return decorated().putAll(map);
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multimap/AbstractSetValuedMap.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multimap/AbstractSetValuedMap.java b/src/main/java/org/apache/commons/collections4/multimap/AbstractSetValuedMap.java
index 96c3140..88704f6 100644
--- a/src/main/java/org/apache/commons/collections4/multimap/AbstractSetValuedMap.java
+++ b/src/main/java/org/apache/commons/collections4/multimap/AbstractSetValuedMap.java
@@ -48,7 +48,7 @@ public abstract class AbstractSetValuedMap<K, V> extends AbstractMultiValuedMap<
      * @param map  the map to wrap, must not be null
      * @throws NullPointerException if the map is null
      */
-    protected AbstractSetValuedMap(Map<K, ? extends Set<V>> map) {
+    protected AbstractSetValuedMap(final Map<K, ? extends Set<V>> map) {
         super(map);
     }
 
@@ -95,7 +95,7 @@ public abstract class AbstractSetValuedMap<K, V> extends AbstractMultiValuedMap<
      *   unmodifiable set for no mapping found.
      */
     @Override
-    public Set<V> remove(Object key) {
+    public Set<V> remove(final Object key) {
         return SetUtils.emptyIfNull(getMap().remove(key));
     }
 
@@ -111,7 +111,7 @@ public abstract class AbstractSetValuedMap<K, V> extends AbstractMultiValuedMap<
         }
 
         @Override
-        public boolean equals(Object other) {
+        public boolean equals(final Object other) {
             final Set<V> set = (Set<V>) getMapping();
             if (set == null) {
                 return Collections.emptySet().equals(other);

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multimap/ArrayListValuedHashMap.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multimap/ArrayListValuedHashMap.java b/src/main/java/org/apache/commons/collections4/multimap/ArrayListValuedHashMap.java
index 873dccf..74255e3 100644
--- a/src/main/java/org/apache/commons/collections4/multimap/ArrayListValuedHashMap.java
+++ b/src/main/java/org/apache/commons/collections4/multimap/ArrayListValuedHashMap.java
@@ -74,7 +74,7 @@ public class ArrayListValuedHashMap<K, V> extends AbstractListValuedMap<K, V>
      *
      * @param initialListCapacity  the initial capacity used for value collections
      */
-    public ArrayListValuedHashMap(int initialListCapacity) {
+    public ArrayListValuedHashMap(final int initialListCapacity) {
         this(DEFAULT_INITIAL_MAP_CAPACITY, initialListCapacity);
     }
 
@@ -85,7 +85,7 @@ public class ArrayListValuedHashMap<K, V> extends AbstractListValuedMap<K, V>
      * @param initialMapCapacity  the initial hashmap capacity
      * @param initialListCapacity  the initial capacity used for value collections
      */
-    public ArrayListValuedHashMap(int initialMapCapacity, int initialListCapacity) {
+    public ArrayListValuedHashMap(final int initialMapCapacity, final int initialListCapacity) {
         super(new HashMap<K, ArrayList<V>>(initialMapCapacity));
         this.initialListCapacity = initialListCapacity;
     }
@@ -128,12 +128,12 @@ public class ArrayListValuedHashMap<K, V> extends AbstractListValuedMap<K, V>
     }
 
     // -----------------------------------------------------------------------
-    private void writeObject(ObjectOutputStream oos) throws IOException {
+    private void writeObject(final ObjectOutputStream oos) throws IOException {
         oos.defaultWriteObject();
         doWriteObject(oos);
     }
 
-    private void readObject(ObjectInputStream ois) throws IOException, ClassNotFoundException {
+    private void readObject(final ObjectInputStream ois) throws IOException, ClassNotFoundException {
         ois.defaultReadObject();
         setMap(new HashMap<K, ArrayList<V>>());
         doReadObject(ois);

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multimap/HashSetValuedHashMap.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multimap/HashSetValuedHashMap.java b/src/main/java/org/apache/commons/collections4/multimap/HashSetValuedHashMap.java
index b06ada8..616287d 100644
--- a/src/main/java/org/apache/commons/collections4/multimap/HashSetValuedHashMap.java
+++ b/src/main/java/org/apache/commons/collections4/multimap/HashSetValuedHashMap.java
@@ -73,7 +73,7 @@ public class HashSetValuedHashMap<K, V> extends AbstractSetValuedMap<K, V>
      *
      * @param initialSetCapacity  the initial capacity used for value collections
      */
-    public HashSetValuedHashMap(int initialSetCapacity) {
+    public HashSetValuedHashMap(final int initialSetCapacity) {
         this(DEFAULT_INITIAL_MAP_CAPACITY, initialSetCapacity);
     }
 
@@ -84,7 +84,7 @@ public class HashSetValuedHashMap<K, V> extends AbstractSetValuedMap<K, V>
      * @param initialMapCapacity  the initial hashmap capacity
      * @param initialSetCapacity  the initial capacity used for value collections
      */
-    public HashSetValuedHashMap(int initialMapCapacity, int initialSetCapacity) {
+    public HashSetValuedHashMap(final int initialMapCapacity, final int initialSetCapacity) {
         super(new HashMap<K, HashSet<V>>(initialMapCapacity));
         this.initialSetCapacity = initialSetCapacity;
     }
@@ -116,12 +116,12 @@ public class HashSetValuedHashMap<K, V> extends AbstractSetValuedMap<K, V>
     }
 
     // -----------------------------------------------------------------------
-    private void writeObject(ObjectOutputStream oos) throws IOException {
+    private void writeObject(final ObjectOutputStream oos) throws IOException {
         oos.defaultWriteObject();
         doWriteObject(oos);
     }
 
-    private void readObject(ObjectInputStream ois) throws IOException, ClassNotFoundException {
+    private void readObject(final ObjectInputStream ois) throws IOException, ClassNotFoundException {
         ois.defaultReadObject();
         setMap(new HashMap<K, HashSet<V>>());
         doReadObject(ois);

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMap.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMap.java b/src/main/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMap.java
index 1bd629f..d8932bb 100644
--- a/src/main/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMap.java
+++ b/src/main/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMap.java
@@ -60,7 +60,7 @@ public final class UnmodifiableMultiValuedMap<K, V>
      */
     @SuppressWarnings("unchecked")
     public static <K, V> UnmodifiableMultiValuedMap<K, V> unmodifiableMultiValuedMap(
-            MultiValuedMap<? extends K, ? extends V> map) {
+            final MultiValuedMap<? extends K, ? extends V> map) {
         if (map instanceof Unmodifiable) {
             return (UnmodifiableMultiValuedMap<K, V>) map;
         }

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multiset/AbstractMapMultiSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multiset/AbstractMapMultiSet.java b/src/main/java/org/apache/commons/collections4/multiset/AbstractMapMultiSet.java
index f742854..f6a0cd8 100644
--- a/src/main/java/org/apache/commons/collections4/multiset/AbstractMapMultiSet.java
+++ b/src/main/java/org/apache/commons/collections4/multiset/AbstractMapMultiSet.java
@@ -81,7 +81,7 @@ public abstract class AbstractMapMultiSet<E> extends AbstractMultiSet<E> {
      *
      * @param map the map to wrap
      */
-    protected void setMap(Map<E, MutableInteger> map) {
+    protected void setMap(final Map<E, MutableInteger> map) {
         this.map = map;
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multiset/AbstractMultiSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multiset/AbstractMultiSet.java b/src/main/java/org/apache/commons/collections4/multiset/AbstractMultiSet.java
index 261175f..46624cc 100644
--- a/src/main/java/org/apache/commons/collections4/multiset/AbstractMultiSet.java
+++ b/src/main/java/org/apache/commons/collections4/multiset/AbstractMultiSet.java
@@ -257,7 +257,7 @@ public abstract class AbstractMultiSet<E> extends AbstractCollection<E> implemen
     protected Iterator<E> createUniqueSetIterator() {
         final Transformer<Entry<E>, E> transformer = new Transformer<Entry<E>, E>() {
             @Override
-            public E transform(Entry<E> entry) {
+            public E transform(final Entry<E> entry) {
                 return entry.getElement();
             }
         };
@@ -411,7 +411,7 @@ public abstract class AbstractMultiSet<E> extends AbstractCollection<E> implemen
     protected static abstract class AbstractEntry<E> implements Entry<E> {
 
         @Override
-        public boolean equals(Object object) {
+        public boolean equals(final Object object) {
           if (object instanceof Entry) {
             final Entry<?> other = (Entry<?>) object;
             final E element = this.getElement();

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multiset/AbstractMultiSetDecorator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multiset/AbstractMultiSetDecorator.java b/src/main/java/org/apache/commons/collections4/multiset/AbstractMultiSetDecorator.java
index 4c80eb5..3e2c724 100644
--- a/src/main/java/org/apache/commons/collections4/multiset/AbstractMultiSetDecorator.java
+++ b/src/main/java/org/apache/commons/collections4/multiset/AbstractMultiSetDecorator.java
@@ -79,7 +79,7 @@ public abstract class AbstractMultiSetDecorator<E>
     }
 
     @Override
-    public int setCount(E object, int count) {
+    public int setCount(final E object, final int count) {
         return decorated().setCount(object, count);
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multiset/PredicatedMultiSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multiset/PredicatedMultiSet.java b/src/main/java/org/apache/commons/collections4/multiset/PredicatedMultiSet.java
index b5d1b7e..ff9065c 100644
--- a/src/main/java/org/apache/commons/collections4/multiset/PredicatedMultiSet.java
+++ b/src/main/java/org/apache/commons/collections4/multiset/PredicatedMultiSet.java
@@ -118,7 +118,7 @@ public class PredicatedMultiSet<E> extends PredicatedCollection<E> implements Mu
     }
 
     @Override
-    public int setCount(E object, int count) {
+    public int setCount(final E object, final int count) {
         validate(object);
         return decorated().setCount(object, count);
     }

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multiset/SynchronizedMultiSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multiset/SynchronizedMultiSet.java b/src/main/java/org/apache/commons/collections4/multiset/SynchronizedMultiSet.java
index f069720..8dc48c9 100644
--- a/src/main/java/org/apache/commons/collections4/multiset/SynchronizedMultiSet.java
+++ b/src/main/java/org/apache/commons/collections4/multiset/SynchronizedMultiSet.java
@@ -120,7 +120,7 @@ public class SynchronizedMultiSet<E> extends SynchronizedCollection<E> implement
     }
 
     @Override
-    public int setCount(E object, int count) {
+    public int setCount(final E object, final int count) {
         synchronized (lock) {
             return decorated().setCount(object, count);
         }

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/multiset/UnmodifiableMultiSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/multiset/UnmodifiableMultiSet.java b/src/main/java/org/apache/commons/collections4/multiset/UnmodifiableMultiSet.java
index ddfadc0..3653716 100644
--- a/src/main/java/org/apache/commons/collections4/multiset/UnmodifiableMultiSet.java
+++ b/src/main/java/org/apache/commons/collections4/multiset/UnmodifiableMultiSet.java
@@ -136,7 +136,7 @@ public final class UnmodifiableMultiSet<E>
 
     //-----------------------------------------------------------------------
     @Override
-    public int setCount(E object, int count) {
+    public int setCount(final E object, final int count) {
         throw new UnsupportedOperationException();
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java b/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java
index 8beb531..6aada49 100644
--- a/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java
+++ b/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java
@@ -282,7 +282,7 @@ public class CircularFifoQueue<E> extends AbstractCollection<E>
      * @throws NullPointerException  if the given element is null
      */
     @Override
-    public boolean offer(E element) {
+    public boolean offer(final E element) {
         return add(element);
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/set/AbstractNavigableSetDecorator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/set/AbstractNavigableSetDecorator.java b/src/main/java/org/apache/commons/collections4/set/AbstractNavigableSetDecorator.java
index 01ff705..42fa406 100644
--- a/src/main/java/org/apache/commons/collections4/set/AbstractNavigableSetDecorator.java
+++ b/src/main/java/org/apache/commons/collections4/set/AbstractNavigableSetDecorator.java
@@ -64,22 +64,22 @@ public abstract class AbstractNavigableSetDecorator<E>
     //-----------------------------------------------------------------------
 
     @Override
-    public E lower(E e) {
+    public E lower(final E e) {
         return decorated().lower(e);
     }
 
     @Override
-    public E floor(E e) {
+    public E floor(final E e) {
         return decorated().floor(e);
     }
 
     @Override
-    public E ceiling(E e) {
+    public E ceiling(final E e) {
         return decorated().ceiling(e);
     }
 
     @Override
-    public E higher(E e) {
+    public E higher(final E e) {
         return decorated().higher(e);
     }
 
@@ -104,17 +104,17 @@ public abstract class AbstractNavigableSetDecorator<E>
     }
 
     @Override
-    public NavigableSet<E> subSet(E fromElement, boolean fromInclusive, E toElement, boolean toInclusive) {
+    public NavigableSet<E> subSet(final E fromElement, final boolean fromInclusive, final E toElement, final boolean toInclusive) {
         return decorated().subSet(fromElement, fromInclusive, toElement, toInclusive);
     }
 
     @Override
-    public NavigableSet<E> headSet(E toElement, boolean inclusive) {
+    public NavigableSet<E> headSet(final E toElement, final boolean inclusive) {
         return decorated().headSet(toElement, inclusive);
     }
 
     @Override
-    public NavigableSet<E> tailSet(E fromElement, boolean inclusive) {
+    public NavigableSet<E> tailSet(final E fromElement, final boolean inclusive) {
         return decorated().tailSet(fromElement, inclusive);
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/set/PredicatedNavigableSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/set/PredicatedNavigableSet.java b/src/main/java/org/apache/commons/collections4/set/PredicatedNavigableSet.java
index 3411ceb..5aea154 100644
--- a/src/main/java/org/apache/commons/collections4/set/PredicatedNavigableSet.java
+++ b/src/main/java/org/apache/commons/collections4/set/PredicatedNavigableSet.java
@@ -91,22 +91,22 @@ public class PredicatedNavigableSet<E> extends PredicatedSortedSet<E> implements
     //-----------------------------------------------------------------------
 
     @Override
-    public E lower(E e) {
+    public E lower(final E e) {
         return decorated().lower(e);
     }
 
     @Override
-    public E floor(E e) {
+    public E floor(final E e) {
         return decorated().floor(e);
     }
 
     @Override
-    public E ceiling(E e) {
+    public E ceiling(final E e) {
         return decorated().ceiling(e);
     }
 
     @Override
-    public E higher(E e) {
+    public E higher(final E e) {
         return decorated().higher(e);
     }
 
@@ -131,19 +131,19 @@ public class PredicatedNavigableSet<E> extends PredicatedSortedSet<E> implements
     }
 
     @Override
-    public NavigableSet<E> subSet(E fromElement, boolean fromInclusive, E toElement, boolean toInclusive) {
+    public NavigableSet<E> subSet(final E fromElement, final boolean fromInclusive, final E toElement, final boolean toInclusive) {
         final NavigableSet<E> sub = decorated().subSet(fromElement, fromInclusive, toElement, toInclusive);
         return predicatedNavigableSet(sub, predicate);
     }
 
     @Override
-    public NavigableSet<E> headSet(E toElement, boolean inclusive) {
+    public NavigableSet<E> headSet(final E toElement, final boolean inclusive) {
         final NavigableSet<E> head = decorated().headSet(toElement, inclusive);
         return predicatedNavigableSet(head, predicate);
     }
 
     @Override
-    public NavigableSet<E> tailSet(E fromElement, boolean inclusive) {
+    public NavigableSet<E> tailSet(final E fromElement, final boolean inclusive) {
         final NavigableSet<E> tail = decorated().tailSet(fromElement, inclusive);
         return predicatedNavigableSet(tail, predicate);
     }

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/set/TransformedNavigableSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/set/TransformedNavigableSet.java b/src/main/java/org/apache/commons/collections4/set/TransformedNavigableSet.java
index eea49d7..c7608b1 100644
--- a/src/main/java/org/apache/commons/collections4/set/TransformedNavigableSet.java
+++ b/src/main/java/org/apache/commons/collections4/set/TransformedNavigableSet.java
@@ -112,22 +112,22 @@ public class TransformedNavigableSet<E> extends TransformedSortedSet<E> implemen
     //-----------------------------------------------------------------------
 
     @Override
-    public E lower(E e) {
+    public E lower(final E e) {
         return decorated().lower(e);
     }
 
     @Override
-    public E floor(E e) {
+    public E floor(final E e) {
         return decorated().floor(e);
     }
 
     @Override
-    public E ceiling(E e) {
+    public E ceiling(final E e) {
         return decorated().ceiling(e);
     }
 
     @Override
-    public E higher(E e) {
+    public E higher(final E e) {
         return decorated().higher(e);
     }
 
@@ -152,19 +152,19 @@ public class TransformedNavigableSet<E> extends TransformedSortedSet<E> implemen
     }
 
     @Override
-    public NavigableSet<E> subSet(E fromElement, boolean fromInclusive, E toElement, boolean toInclusive) {
+    public NavigableSet<E> subSet(final E fromElement, final boolean fromInclusive, final E toElement, final boolean toInclusive) {
         final NavigableSet<E> sub = decorated().subSet(fromElement, fromInclusive, toElement, toInclusive);
         return transformingNavigableSet(sub, transformer);
     }
 
     @Override
-    public NavigableSet<E> headSet(E toElement, boolean inclusive) {
+    public NavigableSet<E> headSet(final E toElement, final boolean inclusive) {
         final NavigableSet<E> head = decorated().headSet(toElement, inclusive);
         return transformingNavigableSet(head, transformer);
     }
 
     @Override
-    public NavigableSet<E> tailSet(E fromElement, boolean inclusive) {
+    public NavigableSet<E> tailSet(final E fromElement, final boolean inclusive) {
         final NavigableSet<E> tail = decorated().tailSet(fromElement, inclusive);
         return transformingNavigableSet(tail, transformer);
     }

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/set/UnmodifiableNavigableSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/set/UnmodifiableNavigableSet.java b/src/main/java/org/apache/commons/collections4/set/UnmodifiableNavigableSet.java
index 40b4dc4..6767b99 100644
--- a/src/main/java/org/apache/commons/collections4/set/UnmodifiableNavigableSet.java
+++ b/src/main/java/org/apache/commons/collections4/set/UnmodifiableNavigableSet.java
@@ -136,19 +136,19 @@ public final class UnmodifiableNavigableSet<E>
     }
 
     @Override
-    public NavigableSet<E> subSet(E fromElement, boolean fromInclusive, E toElement, boolean toInclusive) {
+    public NavigableSet<E> subSet(final E fromElement, final boolean fromInclusive, final E toElement, final boolean toInclusive) {
         final NavigableSet<E> sub = decorated().subSet(fromElement, fromInclusive, toElement, toInclusive);
         return unmodifiableNavigableSet(sub);
     }
 
     @Override
-    public NavigableSet<E> headSet(E toElement, boolean inclusive) {
+    public NavigableSet<E> headSet(final E toElement, final boolean inclusive) {
         final NavigableSet<E> head = decorated().headSet(toElement, inclusive);
         return unmodifiableNavigableSet(head);
     }
 
     @Override
-    public NavigableSet<E> tailSet(E fromElement, boolean inclusive) {
+    public NavigableSet<E> tailSet(final E fromElement, final boolean inclusive) {
         final NavigableSet<E> tail = decorated().tailSet(fromElement, inclusive);
         return unmodifiableNavigableSet(tail);
     }

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/main/java/org/apache/commons/collections4/trie/UnmodifiableTrie.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/collections4/trie/UnmodifiableTrie.java b/src/main/java/org/apache/commons/collections4/trie/UnmodifiableTrie.java
index a18f7ef..424adfa 100644
--- a/src/main/java/org/apache/commons/collections4/trie/UnmodifiableTrie.java
+++ b/src/main/java/org/apache/commons/collections4/trie/UnmodifiableTrie.java
@@ -180,12 +180,12 @@ public class UnmodifiableTrie<K, V> implements Trie<K, V>, Serializable, Unmodif
     }
 
     @Override
-    public K nextKey(K key) {
+    public K nextKey(final K key) {
         return delegate.nextKey(key);
     }
 
     @Override
-    public K previousKey(K key) {
+    public K previousKey(final K key) {
         return delegate.previousKey(key);
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java b/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java
index 9553610..60fc30c 100644
--- a/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java
+++ b/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java
@@ -702,7 +702,7 @@ public class CollectionUtilsTest extends MockTestCase {
         final StringBuffer result = new StringBuffer();
         result.append(CollectionUtils.forAllButLastDo(strings, new Closure<String>() {
             @Override
-            public void execute(String input) {
+            public void execute(final String input) {
                 result.append(input+";");
             }
         }));
@@ -712,7 +712,7 @@ public class CollectionUtilsTest extends MockTestCase {
         final StringBuffer resultOne = new StringBuffer();
         resultOne.append(CollectionUtils.forAllButLastDo(oneString, new Closure<String>() {
             @Override
-            public void execute(String input) {
+            public void execute(final String input) {
                 resultOne.append(input+";");
             }
         }));
@@ -1800,7 +1800,7 @@ public class CollectionUtilsTest extends MockTestCase {
 
         Predicate<Integer> lessThanFive = new Predicate<Integer>() {
             @Override
-            public boolean evaluate(Integer object) {
+            public boolean evaluate(final Integer object) {
                 return object < 5;
             }
         };
@@ -1808,7 +1808,7 @@ public class CollectionUtilsTest extends MockTestCase {
 
         Predicate<Integer> lessThanFour = new Predicate<Integer>() {
             @Override
-            public boolean evaluate(Integer object) {
+            public boolean evaluate(final Integer object) {
                 return object < 4;
             }
         };
@@ -1834,12 +1834,12 @@ public class CollectionUtilsTest extends MockTestCase {
         final Collection<String> result = CollectionUtils.removeAll(base, remove, new Equator<String>() {
 
             @Override
-            public boolean equate(String o1, String o2) {
+            public boolean equate(final String o1, final String o2) {
                 return o1.charAt(1) == o2.charAt(1);
             }
 
             @Override
-            public int hash(String o) {
+            public int hash(final String o) {
                 return o.charAt(1);
             }
         });
@@ -1886,12 +1886,12 @@ public class CollectionUtilsTest extends MockTestCase {
         final Collection<String> result = CollectionUtils.retainAll(base, retain, new Equator<String>() {
 
             @Override
-            public boolean equate(String o1, String o2) {
+            public boolean equate(final String o1, final String o2) {
                 return o1.charAt(1) == o2.charAt(1);
             }
 
             @Override
-            public int hash(String o) {
+            public int hash(final String o) {
                 return o.charAt(1);
             }
         });

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/FluentIterableTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/FluentIterableTest.java b/src/test/java/org/apache/commons/collections4/FluentIterableTest.java
index 5b1851f..8c8addc 100644
--- a/src/test/java/org/apache/commons/collections4/FluentIterableTest.java
+++ b/src/test/java/org/apache/commons/collections4/FluentIterableTest.java
@@ -200,7 +200,7 @@ public class FluentIterableTest {
     public void filter() {
         Predicate<Integer> smallerThan3 = new Predicate<Integer>() {
             @Override
-            public boolean evaluate(Integer object) {
+            public boolean evaluate(final Integer object) {
                 return object.intValue() < 3;
             }
         };
@@ -225,7 +225,7 @@ public class FluentIterableTest {
         final AtomicInteger sum = new AtomicInteger(0);
         Closure<Integer> closure = new Closure<Integer>() {
             @Override
-            public void execute(Integer input) {
+            public void execute(final Integer input) {
                 sum.addAndGet(input);
             }
         };
@@ -317,7 +317,7 @@ public class FluentIterableTest {
     public void transform() {
         Transformer<Integer, Integer> squared = new Transformer<Integer, Integer>() {
             @Override
-            public Integer transform(Integer object) {
+            public Integer transform(final Integer object) {
                 return object * object;
             }
         };

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/IterableUtilsTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/IterableUtilsTest.java b/src/test/java/org/apache/commons/collections4/IterableUtilsTest.java
index 7fd4873..33f18fb 100644
--- a/src/test/java/org/apache/commons/collections4/IterableUtilsTest.java
+++ b/src/test/java/org/apache/commons/collections4/IterableUtilsTest.java
@@ -184,12 +184,12 @@ public class IterableUtilsTest {
         final Equator<String> secondLetterEquator = new Equator<String>() {
 
             @Override
-            public boolean equate(String o1, String o2) {
+            public boolean equate(final String o1, final String o2) {
                 return o1.charAt(1) == o2.charAt(1);
             }
 
             @Override
-            public int hash(String o) {
+            public int hash(final String o) {
                 return o.charAt(1);
             }
 
@@ -375,7 +375,7 @@ public class IterableUtilsTest {
 
         Predicate<Integer> lessThanFive = new Predicate<Integer>() {
             @Override
-            public boolean evaluate(Integer object) {
+            public boolean evaluate(final Integer object) {
                 return object < 5;
             }
         };
@@ -383,7 +383,7 @@ public class IterableUtilsTest {
 
         Predicate<Integer> lessThanFour = new Predicate<Integer>() {
             @Override
-            public boolean evaluate(Integer object) {
+            public boolean evaluate(final Integer object) {
                 return object < 4;
             }
         };
@@ -487,7 +487,7 @@ public class IterableUtilsTest {
 
         result = IterableUtils.toString(iterableA, new Transformer<Integer, String>() {
             @Override
-            public String transform(Integer input) {
+            public String transform(final Integer input) {
                 return new Integer(input * 2).toString();
             }
         });
@@ -495,7 +495,7 @@ public class IterableUtilsTest {
 
         result = IterableUtils.toString(new ArrayList<Integer>(), new Transformer<Integer, String>() {
             @Override
-            public String transform(Integer input) {
+            public String transform(final Integer input) {
                 fail("not supposed to reach here");
                 return "";
             }
@@ -504,7 +504,7 @@ public class IterableUtilsTest {
 
         result = IterableUtils.toString(null, new Transformer<Integer, String>() {
             @Override
-            public String transform(Integer input) {
+            public String transform(final Integer input) {
                 fail("not supposed to reach here");
                 return "";
             }
@@ -517,7 +517,7 @@ public class IterableUtilsTest {
         
         Transformer<Integer, String> transformer = new Transformer<Integer, String>() {
             @Override
-            public String transform(Integer input) {
+            public String transform(final Integer input) {
                 return new Integer(input * 2).toString();
             }
         };
@@ -554,7 +554,7 @@ public class IterableUtilsTest {
     public void testToStringWithNullArguments() {
         String result = IterableUtils.toString(null, new Transformer<Integer, String>() {
             @Override
-            public String transform(Integer input) {
+            public String transform(final Integer input) {
                 fail("not supposed to reach here");
                 return "";
             }
@@ -571,7 +571,7 @@ public class IterableUtilsTest {
         try {
             IterableUtils.toString(new ArrayList<Integer>(), new Transformer<Integer, String>() {
                 @Override
-                public String transform(Integer input) {
+                public String transform(final Integer input) {
                     fail("not supposed to reach here");
                     return "";
                 }
@@ -584,7 +584,7 @@ public class IterableUtilsTest {
         try {
             IterableUtils.toString(new ArrayList<Integer>(), new Transformer<Integer, String>() {
                 @Override
-                public String transform(Integer input) {
+                public String transform(final Integer input) {
                     fail("not supposed to reach here");
                     return "";
                 }
@@ -597,7 +597,7 @@ public class IterableUtilsTest {
         try {
             IterableUtils.toString(new ArrayList<Integer>(), new Transformer<Integer, String>() {
                 @Override
-                public String transform(Integer input) {
+                public String transform(final Integer input) {
                     fail("not supposed to reach here");
                     return "";
                 }

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/MapUtilsTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/MapUtilsTest.java b/src/test/java/org/apache/commons/collections4/MapUtilsTest.java
index 1a595a5..e6b9b84 100644
--- a/src/test/java/org/apache/commons/collections4/MapUtilsTest.java
+++ b/src/test/java/org/apache/commons/collections4/MapUtilsTest.java
@@ -824,13 +824,13 @@ public class MapUtilsTest {
         int key;
         String name;
 
-        public X(int key, String name) {
+        public X(final int key, final String name) {
             this.key = key;
             this.name = name;
         }
 
         @Override
-        public int compareTo(X o) {
+        public int compareTo(final X o) {
             return key - o.key | name.compareTo(o.name);
         }
 
@@ -850,7 +850,7 @@ public class MapUtilsTest {
         final MultiValueMap<Integer, X> map = MultiValueMap.multiValueMap(new TreeMap<Integer, Collection<X>>());
         MapUtils.populateMap(map, list, new Transformer<X, Integer>() {
             @Override
-            public Integer transform(X input) {
+            public Integer transform(final X input) {
                 return input.key;
             }
         }, TransformerUtils.<X> nopTransformer());

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/TransformerUtilsTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/TransformerUtilsTest.java b/src/test/java/org/apache/commons/collections4/TransformerUtilsTest.java
index 2979d90..13d881b 100644
--- a/src/test/java/org/apache/commons/collections4/TransformerUtilsTest.java
+++ b/src/test/java/org/apache/commons/collections4/TransformerUtilsTest.java
@@ -249,7 +249,7 @@ public class TransformerUtilsTest {
 
         Predicate<Integer> lessThanFivePredicate = new Predicate<Integer>() {
             @Override
-            public boolean evaluate(Integer value) {
+            public boolean evaluate(final Integer value) {
                 return value < 5;
             }
         };

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/bag/TreeBagTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/bag/TreeBagTest.java b/src/test/java/org/apache/commons/collections4/bag/TreeBagTest.java
index ad99c44..5dc8b3d 100644
--- a/src/test/java/org/apache/commons/collections4/bag/TreeBagTest.java
+++ b/src/test/java/org/apache/commons/collections4/bag/TreeBagTest.java
@@ -77,7 +77,7 @@ public class TreeBagTest<T> extends AbstractSortedBagTest<T> {
         
         final Bag<String> bag2 = new TreeBag<>(new Comparator<String>() {
             @Override
-            public int compare(String o1, String o2) {
+            public int compare(final String o1, final String o2) {
                 return o1.compareTo(o2);
             }
         });

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/collection/PredicatedCollectionBuilderTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/collection/PredicatedCollectionBuilderTest.java b/src/test/java/org/apache/commons/collections4/collection/PredicatedCollectionBuilderTest.java
index ab2f338..0c8c5a8 100644
--- a/src/test/java/org/apache/commons/collections4/collection/PredicatedCollectionBuilderTest.java
+++ b/src/test/java/org/apache/commons/collections4/collection/PredicatedCollectionBuilderTest.java
@@ -137,7 +137,7 @@ public class PredicatedCollectionBuilderTest {
 
     private static class OddPredicate implements Predicate<Integer> {
         @Override
-        public boolean evaluate(Integer value) {
+        public boolean evaluate(final Integer value) {
             return value % 2 == 1;
         }
     }

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/iterators/PeekingIteratorTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/iterators/PeekingIteratorTest.java b/src/test/java/org/apache/commons/collections4/iterators/PeekingIteratorTest.java
index c3191b3..3876861 100644
--- a/src/test/java/org/apache/commons/collections4/iterators/PeekingIteratorTest.java
+++ b/src/test/java/org/apache/commons/collections4/iterators/PeekingIteratorTest.java
@@ -134,7 +134,7 @@ public class PeekingIteratorTest<E> extends AbstractIteratorTest<E> {
         }
     }
 
-    private void validate(Iterator<E> iter, E... items) {
+    private void validate(final Iterator<E> iter, final E... items) {
         for (E x : items) {
             assertTrue(iter.hasNext());
             assertEquals(x, iter.next());

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/iterators/PushbackIteratorTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/iterators/PushbackIteratorTest.java b/src/test/java/org/apache/commons/collections4/iterators/PushbackIteratorTest.java
index 6561704..3b7a9c0 100644
--- a/src/test/java/org/apache/commons/collections4/iterators/PushbackIteratorTest.java
+++ b/src/test/java/org/apache/commons/collections4/iterators/PushbackIteratorTest.java
@@ -109,7 +109,7 @@ public class PushbackIteratorTest<E> extends AbstractIteratorTest<E> {
         validate(iter, "c");
     }
 
-    private void validate(Iterator<E> iter, Object... items) {
+    private void validate(final Iterator<E> iter, final Object... items) {
         for (final Object x : items) {
             assertTrue(iter.hasNext());
             assertEquals(x, iter.next());

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/keyvalue/MultiKeyTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/keyvalue/MultiKeyTest.java b/src/test/java/org/apache/commons/collections4/keyvalue/MultiKeyTest.java
index 500b837..b3dc8b9 100644
--- a/src/test/java/org/apache/commons/collections4/keyvalue/MultiKeyTest.java
+++ b/src/test/java/org/apache/commons/collections4/keyvalue/MultiKeyTest.java
@@ -258,7 +258,7 @@ public class MultiKeyTest extends TestCase {
 
         private static final long serialVersionUID = 1928896152249821416L;
 
-        public DerivedMultiKey(T key1, T key2) {
+        public DerivedMultiKey(final T key1, final T key2) {
             super(key1, key2);
         }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/map/MultiValueMapTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/map/MultiValueMapTest.java b/src/test/java/org/apache/commons/collections4/map/MultiValueMapTest.java
index 097ec2a..64310d3 100644
--- a/src/test/java/org/apache/commons/collections4/map/MultiValueMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/map/MultiValueMapTest.java
@@ -407,7 +407,7 @@ public class MultiValueMapTest<K, V> extends AbstractObjectTest {
         }
     }
 
-    private byte[] serialize(Object object) throws IOException {
+    private byte[] serialize(final Object object) throws IOException {
         ByteArrayOutputStream baos = new ByteArrayOutputStream();
         ObjectOutputStream oos = new ObjectOutputStream(baos);
 
@@ -417,7 +417,7 @@ public class MultiValueMapTest<K, V> extends AbstractObjectTest {
         return baos.toByteArray();
     }
     
-    private Object deserialize(byte[] data) throws IOException, ClassNotFoundException {
+    private Object deserialize(final byte[] data) throws IOException, ClassNotFoundException {
         ByteArrayInputStream bais = new ByteArrayInputStream(data);
         ObjectInputStream iis = new ObjectInputStream(bais);
         

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/map/PassiveExpiringMapTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/map/PassiveExpiringMapTest.java b/src/test/java/org/apache/commons/collections4/map/PassiveExpiringMapTest.java
index c633cec..8da6831 100644
--- a/src/test/java/org/apache/commons/collections4/map/PassiveExpiringMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/map/PassiveExpiringMapTest.java
@@ -234,7 +234,7 @@ public class PassiveExpiringMapTest<K, V> extends AbstractMapTest<K, V> {
                 new PassiveExpiringMap.ConstantTimeToLiveExpirationPolicy<String, String>(1, TimeUnit.SECONDS)), 1000);
     }
 
-    private void validateExpiration(final Map<String, String> map, long timeout) {
+    private void validateExpiration(final Map<String, String> map, final long timeout) {
         map.put("a", "b");
         
         assertNotNull(map.get("a"));

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java b/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java
index 1faaaa7..41d92f7 100644
--- a/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java
@@ -57,7 +57,7 @@ public abstract class AbstractMultiValuedMapTest<K, V> extends AbstractObjectTes
     /** MultiValuedHashMap created by reset(). */
     protected MultiValuedMap<K, V> confirmed;
 
-    public AbstractMultiValuedMapTest(String testName) {
+    public AbstractMultiValuedMapTest(final String testName) {
         super(testName);
     }
 
@@ -146,7 +146,7 @@ public abstract class AbstractMultiValuedMapTest<K, V> extends AbstractObjectTes
         return map;
     }
 
-    protected void addSampleMappings(MultiValuedMap<? super K, ? super V> map) {
+    protected void addSampleMappings(final MultiValuedMap<? super K, ? super V> map) {
         final K[] keys = getSampleKeys();
         final V[] values = getSampleValues();
         for (int i = 0; i < keys.length; i++) {
@@ -168,7 +168,7 @@ public abstract class AbstractMultiValuedMapTest<K, V> extends AbstractObjectTes
         return this.confirmed;
     }
 
-    public void setConfirmed(MultiValuedMap<K, V> map) {
+    public void setConfirmed(final MultiValuedMap<K, V> map) {
         this.confirmed = map;
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/multimap/ArrayListValuedHashMapTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/multimap/ArrayListValuedHashMapTest.java b/src/test/java/org/apache/commons/collections4/multimap/ArrayListValuedHashMapTest.java
index 286c0b3..57b38f6 100644
--- a/src/test/java/org/apache/commons/collections4/multimap/ArrayListValuedHashMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/multimap/ArrayListValuedHashMapTest.java
@@ -32,7 +32,7 @@ import org.apache.commons.collections4.MultiValuedMap;
  */
 public class ArrayListValuedHashMapTest<K, V> extends AbstractMultiValuedMapTest<K, V> {
 
-    public ArrayListValuedHashMapTest(String testName) {
+    public ArrayListValuedHashMapTest(final String testName) {
         super(testName);
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/multimap/HashSetValuedHashMapTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/multimap/HashSetValuedHashMapTest.java b/src/test/java/org/apache/commons/collections4/multimap/HashSetValuedHashMapTest.java
index 61d5538..7455831 100644
--- a/src/test/java/org/apache/commons/collections4/multimap/HashSetValuedHashMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/multimap/HashSetValuedHashMapTest.java
@@ -32,7 +32,7 @@ import org.apache.commons.collections4.SetValuedMap;
  */
 public class HashSetValuedHashMapTest<K, V> extends AbstractMultiValuedMapTest<K, V> {
 
-    public HashSetValuedHashMapTest(String testName) {
+    public HashSetValuedHashMapTest(final String testName) {
         super(testName);
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/multimap/TransformedMultiValuedMapTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/multimap/TransformedMultiValuedMapTest.java b/src/test/java/org/apache/commons/collections4/multimap/TransformedMultiValuedMapTest.java
index 5f16087..0133ee0 100644
--- a/src/test/java/org/apache/commons/collections4/multimap/TransformedMultiValuedMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/multimap/TransformedMultiValuedMapTest.java
@@ -33,7 +33,7 @@ import org.apache.commons.collections4.collection.TransformedCollectionTest;
  */
 public class TransformedMultiValuedMapTest<K, V> extends AbstractMultiValuedMapTest<K, V> {
 
-    public TransformedMultiValuedMapTest(String testName) {
+    public TransformedMultiValuedMapTest(final String testName) {
         super(testName);
     }
 

http://git-wip-us.apache.org/repos/asf/commons-collections/blob/cefe846e/src/test/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMapTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMapTest.java b/src/test/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMapTest.java
index 5851ce0..ad2d776 100644
--- a/src/test/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMapTest.java
@@ -38,7 +38,7 @@ import org.apache.commons.collections4.Unmodifiable;
  */
 public class UnmodifiableMultiValuedMapTest<K, V> extends AbstractMultiValuedMapTest<K, V> {
 
-    public UnmodifiableMultiValuedMapTest(String testName) {
+    public UnmodifiableMultiValuedMapTest(final String testName) {
         super(testName);
     }
 


Mime
View raw message