You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2018/07/07 17:39:17 UTC

commons-collections git commit: Use final.

Repository: commons-collections
Updated Branches:
  refs/heads/master 525d1fd7c -> a528bcb1e


Use final.

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

Branch: refs/heads/master
Commit: a528bcb1ebafb1eeae14fd3e305f3636d6c04717
Parents: 525d1fd
Author: Gary Gregory <ga...@gmail.com>
Authored: Sat Jul 7 11:39:15 2018 -0600
Committer: Gary Gregory <ga...@gmail.com>
Committed: Sat Jul 7 11:39:15 2018 -0600

----------------------------------------------------------------------
 .../org/apache/commons/collections4/map/AbstractMapTest.java   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/commons-collections/blob/a528bcb1/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java b/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java
index cd9a462..c32d60f 100644
--- a/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java
@@ -647,7 +647,7 @@ public abstract class AbstractMapTest<K, V> extends AbstractObjectTest {
         final Object[] values = getSampleValues();
 
         resetEmpty();
-        for (Object value : values) {
+        for (final Object value : values) {
             assertTrue("Empty map must not contain value",
                        !getMap().containsValue(value));
         }
@@ -1203,7 +1203,7 @@ public abstract class AbstractMapTest<K, V> extends AbstractObjectTest {
         resetFull();
         final V[] sampleValues = getSampleValues();
         final Collection<V> values = getMap().values();
-        for (V sampleValue : sampleValues) {
+        for (final V sampleValue : sampleValues) {
             if (map.containsValue(sampleValue)) {
                 int j = 0;  // loop counter prevents infinite loops when remove is broken
                 while (values.contains(sampleValue) && j < 10000) {
@@ -1315,7 +1315,7 @@ public abstract class AbstractMapTest<K, V> extends AbstractObjectTest {
         resetFull();
         final K[] sampleKeys = getSampleKeys();
         final Set<K> keys = getMap().keySet();
-        for (K sampleKey : sampleKeys) {
+        for (final K sampleKey : sampleKeys) {
             try {
                 keys.remove(sampleKey);
             } catch (final UnsupportedOperationException e) {