Better parameter names

src/main/java/org/apache/commons/collections4/bloomfilter/WrappedBloomFilter.java

@@ -30,10 +30,10 @@ public abstract class WrappedBloomFilter implements BloomFilter {
     /**
      * Wraps a Bloom filter.  The wrapped filter is maintained as a reference
      * not a copy.  Changes in one will be reflected in the other.
-     * @param bf The Bloom filter.
+     * @param wrapped The Bloom filter.
      */
-    public WrappedBloomFilter(final BloomFilter bf) {
-        this.wrapped = bf;
+    public WrappedBloomFilter(final BloomFilter wrapped) {
+        this.wrapped = wrapped;
     }
 
     @Override

src/main/java/org/apache/commons/collections4/collection/AbstractCollectionDecorator.java

@@ -155,10 +155,10 @@ public abstract class AbstractCollectionDecorator<E>
      * <p>
      * <b>NOTE:</b> this method should only be used during deserialization
      *
-     * @param coll  the decorated collection
+     * @param collection  the decorated collection
      */
-    protected void setCollection(final Collection<E> coll) {
-        this.collection = coll;
+    protected void setCollection(final Collection<E> collection) {
+        this.collection = collection;
     }
 
     @Override

src/main/java/org/apache/commons/collections4/map/Flat3Map.java

@@ -250,10 +250,10 @@ public class Flat3Map<K, V> implements IterableMap<K, V>, Serializable, Cloneabl
          * As a consequence, all subsequent call to {@link #getKey()},
          * {@link #setValue(Object)} and {@link #getValue()} will fail.
          *
-         * @param flag the new value of the removed flag
+         * @param removed the new value of the removed flag
          */
-        void setRemoved(final boolean flag) {
-            this.removed = flag;
+        void setRemoved(final boolean removed) {
+            this.removed = removed;
         }
 
         @Override

src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java

@@ -693,8 +693,8 @@ public abstract class AbstractMultiValuedMapTest<K, V> extends AbstractObjectTes
 //        assertEquals(expected, actual);
 //    }
 
-    public void setConfirmed(final MultiValuedMap<K, V> map) {
-        this.confirmed = map;
+    public void setConfirmed(final MultiValuedMap<K, V> confirmed) {
+        this.confirmed = confirmed;
     }
 
     @Test

src/test/java/org/apache/commons/collections4/set/EmptySetMutator.java

@@ -32,8 +32,8 @@ final class EmptySetMutator<E> implements CompositeSet.SetMutator<E> {
 
     private final Set<E> contained;
 
-    EmptySetMutator(final Set<E> set) {
-        this.contained = set;
+    EmptySetMutator(final Set<E> contained) {
+        this.contained = contained;
     }
 
     @Override