diff --git a/src/main/java/org/apache/commons/collections4/ArrayUtils.java b/src/main/java/org/apache/commons/collections4/ArrayUtils.java index 61cbb4821..e997a12f5 100644 --- a/src/main/java/org/apache/commons/collections4/ArrayUtils.java +++ b/src/main/java/org/apache/commons/collections4/ArrayUtils.java @@ -37,7 +37,6 @@ package org.apache.commons.collections4; */ final class ArrayUtils { - /** * Don't allow instances. */ diff --git a/src/main/java/org/apache/commons/collections4/CollectionUtils.java b/src/main/java/org/apache/commons/collections4/CollectionUtils.java index 97e187b71..3ae1da873 100644 --- a/src/main/java/org/apache/commons/collections4/CollectionUtils.java +++ b/src/main/java/org/apache/commons/collections4/CollectionUtils.java @@ -1795,7 +1795,6 @@ public class CollectionUtils { return mergedList; } - /** * Returns a {@link Collection} of all the permutations of the input collection. *

diff --git a/src/main/java/org/apache/commons/collections4/ListUtils.java b/src/main/java/org/apache/commons/collections4/ListUtils.java index 054c070a2..f7eb95c5e 100644 --- a/src/main/java/org/apache/commons/collections4/ListUtils.java +++ b/src/main/java/org/apache/commons/collections4/ListUtils.java @@ -63,7 +63,6 @@ public class ListUtils { } } - /** * A helper class used to construct the longest common subsequence. */ diff --git a/src/main/java/org/apache/commons/collections4/QueueUtils.java b/src/main/java/org/apache/commons/collections4/QueueUtils.java index 79ed44dcc..1cdb18d37 100644 --- a/src/main/java/org/apache/commons/collections4/QueueUtils.java +++ b/src/main/java/org/apache/commons/collections4/QueueUtils.java @@ -42,7 +42,6 @@ public class QueueUtils { */ private QueueUtils() {} - /** * Returns a synchronized (thread-safe) queue backed by the given queue. * In order to guarantee serial access, it is critical that all access to the diff --git a/src/main/java/org/apache/commons/collections4/SetUtils.java b/src/main/java/org/apache/commons/collections4/SetUtils.java index 6d7bcd622..a9269952a 100644 --- a/src/main/java/org/apache/commons/collections4/SetUtils.java +++ b/src/main/java/org/apache/commons/collections4/SetUtils.java @@ -198,7 +198,6 @@ public class SetUtils { return set == null ? Collections.emptySet() : set; } - /** * Gets a typed empty unmodifiable Set. * @param the element type diff --git a/src/main/java/org/apache/commons/collections4/SplitMapUtils.java b/src/main/java/org/apache/commons/collections4/SplitMapUtils.java index b1bf9bcaf..4e8cceadd 100644 --- a/src/main/java/org/apache/commons/collections4/SplitMapUtils.java +++ b/src/main/java/org/apache/commons/collections4/SplitMapUtils.java @@ -43,7 +43,6 @@ public class SplitMapUtils { */ private SplitMapUtils() {} - private static final class WrappedGet implements IterableMap, Unmodifiable { private final Get get; @@ -218,7 +217,6 @@ public class SplitMapUtils { } } - /** * Gets the specified {@link Get} as an instance of {@link IterableMap}. * If {@code get} implements {@link IterableMap} directly, no conversion will take place. diff --git a/src/main/java/org/apache/commons/collections4/bag/AbstractBagDecorator.java b/src/main/java/org/apache/commons/collections4/bag/AbstractBagDecorator.java index c971469b3..f9e09f0f7 100644 --- a/src/main/java/org/apache/commons/collections4/bag/AbstractBagDecorator.java +++ b/src/main/java/org/apache/commons/collections4/bag/AbstractBagDecorator.java @@ -73,7 +73,6 @@ public abstract class AbstractBagDecorator return decorated().hashCode(); } - @Override public int getCount(final Object object) { return decorated().getCount(object); diff --git a/src/main/java/org/apache/commons/collections4/bag/AbstractSortedBagDecorator.java b/src/main/java/org/apache/commons/collections4/bag/AbstractSortedBagDecorator.java index 3c429dfbc..dbd319f25 100644 --- a/src/main/java/org/apache/commons/collections4/bag/AbstractSortedBagDecorator.java +++ b/src/main/java/org/apache/commons/collections4/bag/AbstractSortedBagDecorator.java @@ -62,7 +62,6 @@ public abstract class AbstractSortedBagDecorator return (SortedBag) super.decorated(); } - @Override public E first() { return decorated().first(); diff --git a/src/main/java/org/apache/commons/collections4/bag/PredicatedBag.java b/src/main/java/org/apache/commons/collections4/bag/PredicatedBag.java index 485bc5ec9..d30e34734 100644 --- a/src/main/java/org/apache/commons/collections4/bag/PredicatedBag.java +++ b/src/main/java/org/apache/commons/collections4/bag/PredicatedBag.java @@ -101,7 +101,6 @@ public class PredicatedBag extends PredicatedCollection implements Bag return decorated().hashCode(); } - @Override public boolean add(final E object, final int count) { validate(object); diff --git a/src/main/java/org/apache/commons/collections4/bag/PredicatedSortedBag.java b/src/main/java/org/apache/commons/collections4/bag/PredicatedSortedBag.java index a56e239eb..ebbc09899 100644 --- a/src/main/java/org/apache/commons/collections4/bag/PredicatedSortedBag.java +++ b/src/main/java/org/apache/commons/collections4/bag/PredicatedSortedBag.java @@ -89,7 +89,6 @@ public class PredicatedSortedBag extends PredicatedBag implements SortedBa return (SortedBag) super.decorated(); } - @Override public E first() { return decorated().first(); diff --git a/src/main/java/org/apache/commons/collections4/bag/SynchronizedBag.java b/src/main/java/org/apache/commons/collections4/bag/SynchronizedBag.java index 3420f4ddf..91f882240 100644 --- a/src/main/java/org/apache/commons/collections4/bag/SynchronizedBag.java +++ b/src/main/java/org/apache/commons/collections4/bag/SynchronizedBag.java @@ -100,7 +100,6 @@ public class SynchronizedBag extends SynchronizedCollection implements Bag } } - @Override public boolean add(final E object, final int count) { synchronized (lock) { diff --git a/src/main/java/org/apache/commons/collections4/bag/SynchronizedSortedBag.java b/src/main/java/org/apache/commons/collections4/bag/SynchronizedSortedBag.java index 3c226ebfd..9aaeda2a3 100644 --- a/src/main/java/org/apache/commons/collections4/bag/SynchronizedSortedBag.java +++ b/src/main/java/org/apache/commons/collections4/bag/SynchronizedSortedBag.java @@ -83,7 +83,6 @@ public class SynchronizedSortedBag extends SynchronizedBag implements Sort return (SortedBag) decorated(); } - @Override public synchronized E first() { synchronized (lock) { diff --git a/src/main/java/org/apache/commons/collections4/bag/TransformedBag.java b/src/main/java/org/apache/commons/collections4/bag/TransformedBag.java index 7c5bed3e5..755b10eea 100644 --- a/src/main/java/org/apache/commons/collections4/bag/TransformedBag.java +++ b/src/main/java/org/apache/commons/collections4/bag/TransformedBag.java @@ -121,7 +121,6 @@ public class TransformedBag extends TransformedCollection implements Bag extends TransformedCollection implements Bag extends TransformedBag implements Sorted return (SortedBag) decorated(); } - @Override public E first() { return getSortedBag().first(); diff --git a/src/main/java/org/apache/commons/collections4/bag/TreeBag.java b/src/main/java/org/apache/commons/collections4/bag/TreeBag.java index 421b49d3a..c1905a29b 100644 --- a/src/main/java/org/apache/commons/collections4/bag/TreeBag.java +++ b/src/main/java/org/apache/commons/collections4/bag/TreeBag.java @@ -95,7 +95,6 @@ public class TreeBag extends AbstractMapBag implements SortedBag, Seria return super.add(object); } - @Override public E first() { return getMap().firstKey(); diff --git a/src/main/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap.java b/src/main/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap.java index 0c38a97d4..c5becd8d6 100644 --- a/src/main/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap.java +++ b/src/main/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap.java @@ -130,7 +130,6 @@ public class DualTreeBidiMap extends AbstractDualBidiMap return new DualTreeBidiMap<>(normalMap, reverseMap, inverseMap); } - @Override public Comparator comparator() { return ((SortedMap) normalMap).comparator(); @@ -205,7 +204,6 @@ public class DualTreeBidiMap extends AbstractDualBidiMap return inverseBidiMap(); } - @Override public SortedMap headMap(final K toKey) { final SortedMap sub = ((SortedMap) normalMap).headMap(toKey); diff --git a/src/main/java/org/apache/commons/collections4/bidimap/TreeBidiMap.java b/src/main/java/org/apache/commons/collections4/bidimap/TreeBidiMap.java index c6e46fe5e..ec02fc8e9 100644 --- a/src/main/java/org/apache/commons/collections4/bidimap/TreeBidiMap.java +++ b/src/main/java/org/apache/commons/collections4/bidimap/TreeBidiMap.java @@ -672,7 +672,6 @@ public class TreeBidiMap, V extends Comparable> return rval; } - /** * Compares two objects. * diff --git a/src/main/java/org/apache/commons/collections4/collection/AbstractCollectionDecorator.java b/src/main/java/org/apache/commons/collections4/collection/AbstractCollectionDecorator.java index 13fc7588d..b94652841 100644 --- a/src/main/java/org/apache/commons/collections4/collection/AbstractCollectionDecorator.java +++ b/src/main/java/org/apache/commons/collections4/collection/AbstractCollectionDecorator.java @@ -103,7 +103,6 @@ public abstract class AbstractCollectionDecorator this.collection = coll; } - @Override public boolean add(final E object) { return decorated().add(object); diff --git a/src/main/java/org/apache/commons/collections4/collection/IndexedCollection.java b/src/main/java/org/apache/commons/collections4/collection/IndexedCollection.java index 97544b51c..065232f26 100644 --- a/src/main/java/org/apache/commons/collections4/collection/IndexedCollection.java +++ b/src/main/java/org/apache/commons/collections4/collection/IndexedCollection.java @@ -257,7 +257,6 @@ public class IndexedCollection extends AbstractCollectionDecorator { return changed; } - /** * Provides checking for adding the index. * diff --git a/src/main/java/org/apache/commons/collections4/collection/SynchronizedCollection.java b/src/main/java/org/apache/commons/collections4/collection/SynchronizedCollection.java index 399221690..25b69551c 100644 --- a/src/main/java/org/apache/commons/collections4/collection/SynchronizedCollection.java +++ b/src/main/java/org/apache/commons/collections4/collection/SynchronizedCollection.java @@ -96,7 +96,6 @@ public class SynchronizedCollection implements Collection, Serializable { return collection; } - @Override public boolean add(final E object) { synchronized (lock) { diff --git a/src/main/java/org/apache/commons/collections4/iterators/AbstractListIteratorDecorator.java b/src/main/java/org/apache/commons/collections4/iterators/AbstractListIteratorDecorator.java index 44ddcbfdb..609cf9407 100644 --- a/src/main/java/org/apache/commons/collections4/iterators/AbstractListIteratorDecorator.java +++ b/src/main/java/org/apache/commons/collections4/iterators/AbstractListIteratorDecorator.java @@ -52,7 +52,6 @@ public class AbstractListIteratorDecorator implements ListIterator { return iterator; } - /** {@inheritDoc} */ @Override public boolean hasNext() { diff --git a/src/main/java/org/apache/commons/collections4/iterators/AbstractMapIteratorDecorator.java b/src/main/java/org/apache/commons/collections4/iterators/AbstractMapIteratorDecorator.java index 1d8583ba8..192d668d4 100644 --- a/src/main/java/org/apache/commons/collections4/iterators/AbstractMapIteratorDecorator.java +++ b/src/main/java/org/apache/commons/collections4/iterators/AbstractMapIteratorDecorator.java @@ -53,7 +53,6 @@ public class AbstractMapIteratorDecorator implements MapIterator { return iterator; } - /** {@inheritDoc} */ @Override public boolean hasNext() { diff --git a/src/main/java/org/apache/commons/collections4/iterators/AbstractOrderedMapIteratorDecorator.java b/src/main/java/org/apache/commons/collections4/iterators/AbstractOrderedMapIteratorDecorator.java index 5738ac194..1a8e9f7bf 100644 --- a/src/main/java/org/apache/commons/collections4/iterators/AbstractOrderedMapIteratorDecorator.java +++ b/src/main/java/org/apache/commons/collections4/iterators/AbstractOrderedMapIteratorDecorator.java @@ -53,7 +53,6 @@ public class AbstractOrderedMapIteratorDecorator implements OrderedMapIter return iterator; } - /** {@inheritDoc} */ @Override public boolean hasNext() { diff --git a/src/main/java/org/apache/commons/collections4/iterators/BoundedIterator.java b/src/main/java/org/apache/commons/collections4/iterators/BoundedIterator.java index cf1915d13..1b4b786a3 100644 --- a/src/main/java/org/apache/commons/collections4/iterators/BoundedIterator.java +++ b/src/main/java/org/apache/commons/collections4/iterators/BoundedIterator.java @@ -49,7 +49,6 @@ public class BoundedIterator implements Iterator { /** The position of the current element */ private long pos; - /** * Decorates the specified iterator to return at most the given number of elements, * skipping all elements until the iterator reaches the position at {@code offset}. @@ -88,7 +87,6 @@ public class BoundedIterator implements Iterator { } } - @Override public boolean hasNext() { if (!checkBounds()) { diff --git a/src/main/java/org/apache/commons/collections4/iterators/CollatingIterator.java b/src/main/java/org/apache/commons/collections4/iterators/CollatingIterator.java index 7bf975a27..0b2b4e123 100644 --- a/src/main/java/org/apache/commons/collections4/iterators/CollatingIterator.java +++ b/src/main/java/org/apache/commons/collections4/iterators/CollatingIterator.java @@ -27,7 +27,6 @@ import java.util.Objects; import org.apache.commons.collections4.list.UnmodifiableList; - /** * Provides an ordered iteration over the elements contained in a collection of * ordered Iterators. diff --git a/src/main/java/org/apache/commons/collections4/iterators/LazyIteratorChain.java b/src/main/java/org/apache/commons/collections4/iterators/LazyIteratorChain.java index 94811a9b1..4aa37815b 100644 --- a/src/main/java/org/apache/commons/collections4/iterators/LazyIteratorChain.java +++ b/src/main/java/org/apache/commons/collections4/iterators/LazyIteratorChain.java @@ -63,7 +63,6 @@ public abstract class LazyIteratorChain implements Iterator { */ private Iterator lastUsedIterator; - /** * Gets the next iterator after the previous one has been exhausted. *

@@ -100,7 +99,6 @@ public abstract class LazyIteratorChain implements Iterator { } } - /** * Return true if any Iterator in the chain has a remaining element. * diff --git a/src/main/java/org/apache/commons/collections4/iterators/PeekingIterator.java b/src/main/java/org/apache/commons/collections4/iterators/PeekingIterator.java index ce38757eb..7a37bd9d2 100644 --- a/src/main/java/org/apache/commons/collections4/iterators/PeekingIterator.java +++ b/src/main/java/org/apache/commons/collections4/iterators/PeekingIterator.java @@ -64,7 +64,6 @@ public class PeekingIterator implements Iterator { return new PeekingIterator<>(iterator); } - /** * Constructor. * diff --git a/src/main/java/org/apache/commons/collections4/iterators/PushbackIterator.java b/src/main/java/org/apache/commons/collections4/iterators/PushbackIterator.java index 467a82107..b8b5efdc9 100644 --- a/src/main/java/org/apache/commons/collections4/iterators/PushbackIterator.java +++ b/src/main/java/org/apache/commons/collections4/iterators/PushbackIterator.java @@ -61,7 +61,6 @@ public class PushbackIterator implements Iterator { return new PushbackIterator<>(iterator); } - /** * Constructor. * diff --git a/src/main/java/org/apache/commons/collections4/iterators/SkippingIterator.java b/src/main/java/org/apache/commons/collections4/iterators/SkippingIterator.java index 857c52d3c..b3ace8563 100644 --- a/src/main/java/org/apache/commons/collections4/iterators/SkippingIterator.java +++ b/src/main/java/org/apache/commons/collections4/iterators/SkippingIterator.java @@ -37,7 +37,6 @@ public class SkippingIterator extends AbstractIteratorDecorator { /** The position of the current element */ private long pos; - /** * Decorates the specified iterator to skip all elements until the iterator * reaches the position at {@code offset}. @@ -71,7 +70,6 @@ public class SkippingIterator extends AbstractIteratorDecorator { } } - @Override public E next() { final E next = super.next(); diff --git a/src/main/java/org/apache/commons/collections4/keyvalue/AbstractMapEntryDecorator.java b/src/main/java/org/apache/commons/collections4/keyvalue/AbstractMapEntryDecorator.java index 1cdd37ad1..09d9fe18b 100644 --- a/src/main/java/org/apache/commons/collections4/keyvalue/AbstractMapEntryDecorator.java +++ b/src/main/java/org/apache/commons/collections4/keyvalue/AbstractMapEntryDecorator.java @@ -53,7 +53,6 @@ public abstract class AbstractMapEntryDecorator implements Map.Entry return entry; } - @Override public K getKey() { return entry.getKey(); diff --git a/src/main/java/org/apache/commons/collections4/keyvalue/MultiKey.java b/src/main/java/org/apache/commons/collections4/keyvalue/MultiKey.java index 2c39bf5c4..514eef693 100644 --- a/src/main/java/org/apache/commons/collections4/keyvalue/MultiKey.java +++ b/src/main/java/org/apache/commons/collections4/keyvalue/MultiKey.java @@ -131,7 +131,6 @@ public class MultiKey implements Serializable { this(newArray(key1, key2), false); } - /** * Constructor taking three keys. *

diff --git a/src/main/java/org/apache/commons/collections4/list/AbstractLinkedList.java b/src/main/java/org/apache/commons/collections4/list/AbstractLinkedList.java index ce1b539a7..83267b6be 100644 --- a/src/main/java/org/apache/commons/collections4/list/AbstractLinkedList.java +++ b/src/main/java/org/apache/commons/collections4/list/AbstractLinkedList.java @@ -99,7 +99,6 @@ public abstract class AbstractLinkedList implements List { header = createHeaderNode(); } - @Override public int size() { return size; @@ -116,7 +115,6 @@ public abstract class AbstractLinkedList implements List { return node.getValue(); } - @Override public Iterator iterator() { return listIterator(); @@ -132,7 +130,6 @@ public abstract class AbstractLinkedList implements List { return new LinkedListIterator<>(this, fromIndex); } - @Override public int indexOf(final Object value) { int i = 0; @@ -172,7 +169,6 @@ public abstract class AbstractLinkedList implements List { return true; } - @Override public Object[] toArray() { return toArray(new Object[size]); @@ -210,7 +206,6 @@ public abstract class AbstractLinkedList implements List { return new LinkedSubList<>(this, fromIndexInclusive, toIndexExclusive); } - @Override public boolean add(final E value) { addLast(value); @@ -237,7 +232,6 @@ public abstract class AbstractLinkedList implements List { return true; } - @Override public E remove(final int index) { final Node node = getNode(index, false); @@ -279,7 +273,6 @@ public abstract class AbstractLinkedList implements List { return modified; } - /** * {@inheritDoc} *

@@ -315,7 +308,6 @@ public abstract class AbstractLinkedList implements List { removeAllNodes(); } - public E getFirst() { final Node node = header.next; if (node == header) { diff --git a/src/main/java/org/apache/commons/collections4/list/AbstractListDecorator.java b/src/main/java/org/apache/commons/collections4/list/AbstractListDecorator.java index 6c8d46945..23ee15065 100644 --- a/src/main/java/org/apache/commons/collections4/list/AbstractListDecorator.java +++ b/src/main/java/org/apache/commons/collections4/list/AbstractListDecorator.java @@ -74,7 +74,6 @@ public abstract class AbstractListDecorator extends AbstractCollectionDecorat return decorated().hashCode(); } - @Override public void add(final int index, final E object) { decorated().add(index, object); diff --git a/src/main/java/org/apache/commons/collections4/list/PredicatedList.java b/src/main/java/org/apache/commons/collections4/list/PredicatedList.java index fa2299136..7a64c8d9f 100644 --- a/src/main/java/org/apache/commons/collections4/list/PredicatedList.java +++ b/src/main/java/org/apache/commons/collections4/list/PredicatedList.java @@ -105,7 +105,6 @@ public class PredicatedList extends PredicatedCollection implements List extends PredicatedCollection implements List extends TransformedCollection implements List return decorated().hashCode(); } - @Override public E get(final int index) { return getList().get(index); @@ -145,7 +144,6 @@ public class TransformedList extends TransformedCollection implements List return getList().remove(index); } - @Override public void add(final int index, E object) { object = transform(object); 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 711e3bd04..fc4dbad10 100644 --- a/src/main/java/org/apache/commons/collections4/map/AbstractReferenceMap.java +++ b/src/main/java/org/apache/commons/collections4/map/AbstractReferenceMap.java @@ -255,7 +255,6 @@ public abstract class AbstractReferenceMap extends AbstractHashedMap return entry.getValue(); } - /** * Puts a key-value mapping into this map. * Neither the key nor the value may be null. diff --git a/src/main/java/org/apache/commons/collections4/map/StaticBucketMap.java b/src/main/java/org/apache/commons/collections4/map/StaticBucketMap.java index 2f6bb707e..a1a273e55 100644 --- a/src/main/java/org/apache/commons/collections4/map/StaticBucketMap.java +++ b/src/main/java/org/apache/commons/collections4/map/StaticBucketMap.java @@ -657,7 +657,6 @@ public final class StaticBucketMap extends AbstractIterableMap { } - private final class Values extends AbstractCollection { @Override 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 c06098544..08795a850 100644 --- a/src/main/java/org/apache/commons/collections4/multiset/AbstractMultiSetDecorator.java +++ b/src/main/java/org/apache/commons/collections4/multiset/AbstractMultiSetDecorator.java @@ -72,7 +72,6 @@ public abstract class AbstractMultiSetDecorator return decorated().hashCode(); } - @Override public int getCount(final Object object) { return decorated().getCount(object); 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 668a9db93..c7e55ce27 100644 --- a/src/main/java/org/apache/commons/collections4/multiset/PredicatedMultiSet.java +++ b/src/main/java/org/apache/commons/collections4/multiset/PredicatedMultiSet.java @@ -101,7 +101,6 @@ public class PredicatedMultiSet extends PredicatedCollection implements Mu return decorated().hashCode(); } - @Override public int add(final E object, final int count) { validate(object); 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 f522897ca..0093f6af1 100644 --- a/src/main/java/org/apache/commons/collections4/multiset/SynchronizedMultiSet.java +++ b/src/main/java/org/apache/commons/collections4/multiset/SynchronizedMultiSet.java @@ -97,7 +97,6 @@ public class SynchronizedMultiSet extends SynchronizedCollection implement } } - @Override public int add(final E object, final int count) { synchronized (lock) { diff --git a/src/main/java/org/apache/commons/collections4/queue/AbstractQueueDecorator.java b/src/main/java/org/apache/commons/collections4/queue/AbstractQueueDecorator.java index 6ffebdaa8..aa916951f 100644 --- a/src/main/java/org/apache/commons/collections4/queue/AbstractQueueDecorator.java +++ b/src/main/java/org/apache/commons/collections4/queue/AbstractQueueDecorator.java @@ -68,7 +68,6 @@ public abstract class AbstractQueueDecorator extends AbstractCollectionDecora return (Queue) super.decorated(); } - @Override public boolean offer(final E obj) { return decorated().offer(obj); 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 f79fc111e..a96b86c6f 100644 --- a/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java +++ b/src/main/java/org/apache/commons/collections4/queue/CircularFifoQueue.java @@ -272,7 +272,6 @@ public class CircularFifoQueue extends AbstractCollection return elements[idx]; } - /** * Adds the given element to this queue. If the queue is full, the least recently added * element is discarded so that a new element can be inserted. diff --git a/src/main/java/org/apache/commons/collections4/queue/PredicatedQueue.java b/src/main/java/org/apache/commons/collections4/queue/PredicatedQueue.java index b5b031a47..8d8f51f2f 100644 --- a/src/main/java/org/apache/commons/collections4/queue/PredicatedQueue.java +++ b/src/main/java/org/apache/commons/collections4/queue/PredicatedQueue.java @@ -85,7 +85,6 @@ public class PredicatedQueue extends PredicatedCollection implements Queue return (Queue) super.decorated(); } - /** * Override to validate the object being added to ensure it matches * the predicate. diff --git a/src/main/java/org/apache/commons/collections4/queue/TransformedQueue.java b/src/main/java/org/apache/commons/collections4/queue/TransformedQueue.java index 710a25621..e716fd8ef 100644 --- a/src/main/java/org/apache/commons/collections4/queue/TransformedQueue.java +++ b/src/main/java/org/apache/commons/collections4/queue/TransformedQueue.java @@ -109,7 +109,6 @@ public class TransformedQueue extends TransformedCollection implements Que return (Queue) decorated(); } - @Override public boolean offer(final E obj) { return getQueue().offer(transform(obj)); diff --git a/src/main/java/org/apache/commons/collections4/queue/UnmodifiableQueue.java b/src/main/java/org/apache/commons/collections4/queue/UnmodifiableQueue.java index c8dc50973..219e9d774 100644 --- a/src/main/java/org/apache/commons/collections4/queue/UnmodifiableQueue.java +++ b/src/main/java/org/apache/commons/collections4/queue/UnmodifiableQueue.java @@ -140,7 +140,6 @@ public final class UnmodifiableQueue throw new UnsupportedOperationException(); } - @Override public boolean offer(final E obj) { throw new UnsupportedOperationException(); diff --git a/src/main/java/org/apache/commons/collections4/sequence/SequencesComparator.java b/src/main/java/org/apache/commons/collections4/sequence/SequencesComparator.java index 4ada1916e..83a7f72c8 100644 --- a/src/main/java/org/apache/commons/collections4/sequence/SequencesComparator.java +++ b/src/main/java/org/apache/commons/collections4/sequence/SequencesComparator.java @@ -237,7 +237,6 @@ public class SequencesComparator { throw new IllegalStateException("Internal Error"); } - /** * Build an edit script. * 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 92f13d2fd..0017579f8 100644 --- a/src/main/java/org/apache/commons/collections4/set/AbstractNavigableSetDecorator.java +++ b/src/main/java/org/apache/commons/collections4/set/AbstractNavigableSetDecorator.java @@ -61,7 +61,6 @@ public abstract class AbstractNavigableSetDecorator return (NavigableSet) super.decorated(); } - @Override public E lower(final E e) { return decorated().lower(e); 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 d27477216..c0422264e 100644 --- a/src/main/java/org/apache/commons/collections4/set/PredicatedNavigableSet.java +++ b/src/main/java/org/apache/commons/collections4/set/PredicatedNavigableSet.java @@ -90,7 +90,6 @@ public class PredicatedNavigableSet extends PredicatedSortedSet implements return (NavigableSet) super.decorated(); } - @Override public E lower(final E e) { return decorated().lower(e); 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 82f0740e8..b92937c04 100644 --- a/src/main/java/org/apache/commons/collections4/set/TransformedNavigableSet.java +++ b/src/main/java/org/apache/commons/collections4/set/TransformedNavigableSet.java @@ -110,7 +110,6 @@ public class TransformedNavigableSet extends TransformedSortedSet implemen return (NavigableSet) super.decorated(); } - @Override public E lower(final E e) { return decorated().lower(e); diff --git a/src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java b/src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java index 62ab39f4f..af2c7eb9d 100644 --- a/src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java +++ b/src/main/java/org/apache/commons/collections4/trie/AbstractPatriciaTrie.java @@ -769,7 +769,6 @@ public abstract class AbstractPatriciaTrie extends AbstractBitwiseTrie comparator() { return getKeyAnalyzer(); @@ -1387,7 +1386,6 @@ public abstract class AbstractPatriciaTrie extends AbstractBitwiseTrie extends AbstractBitwiseTrie implements Trie, Serializable, Unmodif this.delegate = tmpTrie; } - @Override public Set> entrySet() { return Collections.unmodifiableSet(delegate.entrySet()); diff --git a/src/test/java/org/apache/commons/collections4/MapUtilsTest.java b/src/test/java/org/apache/commons/collections4/MapUtilsTest.java index 2f8083aed..b632a5724 100644 --- a/src/test/java/org/apache/commons/collections4/MapUtilsTest.java +++ b/src/test/java/org/apache/commons/collections4/MapUtilsTest.java @@ -1016,7 +1016,6 @@ public class MapUtilsTest { assertEquals(1.0, MapUtils.getDouble(in, "noKey", 1.0), 0); assertEquals(1.0, MapUtils.getDouble(in, "noKey", key -> 1.0), 0); - final Map inStr = new HashMap<>(); final char decimalSeparator = getDecimalSeparator(); inStr.put("str1", "2" + decimalSeparator + "0"); @@ -1127,7 +1126,6 @@ public class MapUtilsTest { assertEquals(val, MapUtils.getByte(in, "noKey", val), 0); assertEquals(val, MapUtils.getByte(in, "noKey", key -> val), 0); - final Map inStr = new HashMap<>(); inStr.put("str1", "100"); diff --git a/src/test/java/org/apache/commons/collections4/TrieUtilsTest.java b/src/test/java/org/apache/commons/collections4/TrieUtilsTest.java index 6a8cc3cc2..ec6ab5729 100644 --- a/src/test/java/org/apache/commons/collections4/TrieUtilsTest.java +++ b/src/test/java/org/apache/commons/collections4/TrieUtilsTest.java @@ -41,4 +41,3 @@ public class TrieUtilsTest { } - diff --git a/src/test/java/org/apache/commons/collections4/bag/AbstractSortedBagTest.java b/src/test/java/org/apache/commons/collections4/bag/AbstractSortedBagTest.java index 88f6cbc4e..e4f66f946 100644 --- a/src/test/java/org/apache/commons/collections4/bag/AbstractSortedBagTest.java +++ b/src/test/java/org/apache/commons/collections4/bag/AbstractSortedBagTest.java @@ -99,7 +99,6 @@ public abstract class AbstractSortedBagTest extends AbstractBagTest { return new TreeBag<>(); } - @Override public void resetEmpty() { this.setCollection(CollectionSortedBag.collectionSortedBag(makeObject())); @@ -149,6 +148,5 @@ public abstract class AbstractSortedBagTest extends AbstractBagTest { return (SortedBag) super.getCollection(); } - // TODO: Add the SortedBag tests! } diff --git a/src/test/java/org/apache/commons/collections4/bag/CollectionBagTest.java b/src/test/java/org/apache/commons/collections4/bag/CollectionBagTest.java index d88cb8126..2fa2d8634 100644 --- a/src/test/java/org/apache/commons/collections4/bag/CollectionBagTest.java +++ b/src/test/java/org/apache/commons/collections4/bag/CollectionBagTest.java @@ -46,7 +46,6 @@ public class CollectionBagTest extends AbstractCollectionTest { super(CollectionBagTest.class.getSimpleName()); } - @Override public Bag makeObject() { return CollectionBag.collectionBag(new HashBag<>()); @@ -91,7 +90,6 @@ public class CollectionBagTest extends AbstractCollectionTest { // writeExternalFormToDisk((java.io.Serializable) getCollection(), "src/test/resources/data/test/CollectionBag.fullCollection.version4.obj"); // } - /** * Compares the current serialized form of the Bag * against the canonical version in SCM. diff --git a/src/test/java/org/apache/commons/collections4/bag/CollectionSortedBagTest.java b/src/test/java/org/apache/commons/collections4/bag/CollectionSortedBagTest.java index e65c2db8d..5861a9a42 100644 --- a/src/test/java/org/apache/commons/collections4/bag/CollectionSortedBagTest.java +++ b/src/test/java/org/apache/commons/collections4/bag/CollectionSortedBagTest.java @@ -46,7 +46,6 @@ public class CollectionSortedBagTest extends AbstractCollectionTest { super(CollectionSortedBagTest.class.getSimpleName()); } - /** * Overridden because SortedBags don't allow null elements (normally). * @return false @@ -110,7 +109,6 @@ public class CollectionSortedBagTest extends AbstractCollectionTest { return (T[]) elements; } - @Override public String getCompatibilityVersion() { return "4"; @@ -123,7 +121,6 @@ public class CollectionSortedBagTest extends AbstractCollectionTest { // writeExternalFormToDisk((java.io.Serializable) getCollection(), "src/test/resources/data/test/CollectionSortedBag.fullCollection.version4.obj"); // } - /** * Compare the current serialized form of the Bag * against the canonical version in SCM. diff --git a/src/test/java/org/apache/commons/collections4/bag/SynchronizedBagTest.java b/src/test/java/org/apache/commons/collections4/bag/SynchronizedBagTest.java index 5777ef2ea..e6f5a6079 100644 --- a/src/test/java/org/apache/commons/collections4/bag/SynchronizedBagTest.java +++ b/src/test/java/org/apache/commons/collections4/bag/SynchronizedBagTest.java @@ -30,7 +30,6 @@ public class SynchronizedBagTest extends AbstractBagTest { super(SynchronizedBagTest.class.getSimpleName()); } - @Override public Bag makeObject() { return SynchronizedBag.synchronizedBag(new HashBag<>()); diff --git a/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap2Test.java b/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap2Test.java index fd7516d20..bfa9805ef 100644 --- a/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap2Test.java +++ b/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap2Test.java @@ -157,7 +157,6 @@ public class DualTreeBidiMap2Test, V extends Comparable< return new String[] { recursiveTest }; } - // public void testCreate() throws Exception { // resetEmpty(); // writeExternalFormToDisk((java.io.Serializable) map, "src/test/resources/data/test/DualTreeBidiMap.emptyCollection.version4.Test2.obj"); diff --git a/src/test/java/org/apache/commons/collections4/comparators/AbstractComparatorTest.java b/src/test/java/org/apache/commons/collections4/comparators/AbstractComparatorTest.java index 0bd179d00..34cd50d30 100644 --- a/src/test/java/org/apache/commons/collections4/comparators/AbstractComparatorTest.java +++ b/src/test/java/org/apache/commons/collections4/comparators/AbstractComparatorTest.java @@ -48,7 +48,6 @@ public abstract class AbstractComparatorTest extends AbstractObjectTest { super(testName); } - /** * Implement this method to return a list of sorted objects. * @@ -204,7 +203,6 @@ public abstract class AbstractComparatorTest extends AbstractObjectTest { } } - // make sure the canonical form produces the ordering we currently // expect final List randomList = getComparableObjectsOrdered(); diff --git a/src/test/java/org/apache/commons/collections4/functors/AbstractCompositePredicateTest.java b/src/test/java/org/apache/commons/collections4/functors/AbstractCompositePredicateTest.java index d9217604c..65a9a5f25 100644 --- a/src/test/java/org/apache/commons/collections4/functors/AbstractCompositePredicateTest.java +++ b/src/test/java/org/apache/commons/collections4/functors/AbstractCompositePredicateTest.java @@ -124,7 +124,6 @@ public abstract class AbstractCompositePredicateTest extends AbstractMockPred assertThrows(NullPointerException.class, () -> getPredicateInstance(new Predicate[] { null, null })); } - /** * Tests {@code getInstance} with a null predicate collection */ diff --git a/src/test/java/org/apache/commons/collections4/functors/ComparatorPredicateTest.java b/src/test/java/org/apache/commons/collections4/functors/ComparatorPredicateTest.java index 77a11bd89..0248403a3 100644 --- a/src/test/java/org/apache/commons/collections4/functors/ComparatorPredicateTest.java +++ b/src/test/java/org/apache/commons/collections4/functors/ComparatorPredicateTest.java @@ -24,7 +24,6 @@ import org.apache.commons.collections4.Predicate; import org.apache.commons.collections4.functors.ComparatorPredicate.Criterion; import org.junit.jupiter.api.Test; - public class ComparatorPredicateTest extends AbstractPredicateTest { private static final class TestComparator> implements Comparator { @Override diff --git a/src/test/java/org/apache/commons/collections4/functors/EqualPredicateTest.java b/src/test/java/org/apache/commons/collections4/functors/EqualPredicateTest.java index 5243725c1..45d411db3 100644 --- a/src/test/java/org/apache/commons/collections4/functors/EqualPredicateTest.java +++ b/src/test/java/org/apache/commons/collections4/functors/EqualPredicateTest.java @@ -23,7 +23,6 @@ import static org.junit.jupiter.api.Assertions.assertSame; import org.apache.commons.collections4.Predicate; import org.junit.jupiter.api.Test; - public class EqualPredicateTest extends AbstractPredicateTest { private static final EqualsTestObject FALSE_OBJECT = new EqualsTestObject(false); private static final EqualsTestObject TRUE_OBJECT = new EqualsTestObject(true); diff --git a/src/test/java/org/apache/commons/collections4/iterators/NodeListIteratorTest.java b/src/test/java/org/apache/commons/collections4/iterators/NodeListIteratorTest.java index 620ec1e3e..3e9e12fc5 100644 --- a/src/test/java/org/apache/commons/collections4/iterators/NodeListIteratorTest.java +++ b/src/test/java/org/apache/commons/collections4/iterators/NodeListIteratorTest.java @@ -55,7 +55,6 @@ public class NodeListIteratorTest extends AbstractIteratorTest { // Default: use standard constr. createIteratorWithStandardConstr = true; - // create mocked Node Instances and fill Node[] to be used by test cases final Node node1 = createMock(Element.class); final Node node2 = createMock(Element.class); diff --git a/src/test/java/org/apache/commons/collections4/iterators/PermutationIteratorTest.java b/src/test/java/org/apache/commons/collections4/iterators/PermutationIteratorTest.java index ac65d2e51..8d7ac8d19 100644 --- a/src/test/java/org/apache/commons/collections4/iterators/PermutationIteratorTest.java +++ b/src/test/java/org/apache/commons/collections4/iterators/PermutationIteratorTest.java @@ -53,7 +53,6 @@ public class PermutationIteratorTest extends AbstractIteratorTest extends AbstractCollectionTest { "Exhausted iterator should raise NoSuchElement"); } - /** * Tests the {@link ListIterator#add(Object)} method of the list * iterator. 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 5d19ce181..5f23d8773 100644 --- a/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java +++ b/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java @@ -681,7 +681,6 @@ public abstract class AbstractMapTest extends AbstractObjectTest { verify(); } - /** * Tests Map.equals(Object) */ @@ -1750,7 +1749,6 @@ public abstract class AbstractMapTest extends AbstractObjectTest { } } - /** * Bulk test {@link Map#keySet()}. This method runs through all of * the tests in {@link AbstractSetTest}. @@ -1942,7 +1940,6 @@ public abstract class AbstractMapTest extends AbstractObjectTest { // the value equal to the value returned from the values iterator. } - /** * Resets the {@link #map}, {@link #entrySet}, {@link #keySet}, * {@link #values} and {@link #confirmed} fields to empty. diff --git a/src/test/java/org/apache/commons/collections4/map/LazyMapTest.java b/src/test/java/org/apache/commons/collections4/map/LazyMapTest.java index 2f74d2652..f7da0a7df 100644 --- a/src/test/java/org/apache/commons/collections4/map/LazyMapTest.java +++ b/src/test/java/org/apache/commons/collections4/map/LazyMapTest.java @@ -83,7 +83,6 @@ public class LazyMapTest extends AbstractIterableMapTest { assertEquals(1, map.size()); } - @Override public String getCompatibilityVersion() { return "4"; 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 02f8ecd49..d3b1292b4 100644 --- a/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java +++ b/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java @@ -65,7 +65,6 @@ public abstract class AbstractMultiValuedMapTest extends AbstractObjectTes /** MultiValuedHashMap created by reset(). */ protected MultiValuedMap confirmed; - public AbstractMultiValuedMapTest(final String testName) { super(testName); } diff --git a/src/test/java/org/apache/commons/collections4/multiset/AbstractMultiSetTest.java b/src/test/java/org/apache/commons/collections4/multiset/AbstractMultiSetTest.java index 331b11e47..a72652bf7 100644 --- a/src/test/java/org/apache/commons/collections4/multiset/AbstractMultiSetTest.java +++ b/src/test/java/org/apache/commons/collections4/multiset/AbstractMultiSetTest.java @@ -110,7 +110,6 @@ public abstract class AbstractMultiSetTest extends AbstractCollectionTest return multiset; } - @Override public void resetEmpty() { this.setCollection(makeObject()); @@ -615,7 +614,6 @@ public abstract class AbstractMultiSetTest extends AbstractCollectionTest assertEquals(total, multiset2.hashCode()); } - /** * Bulk test {@link MultiSet#uniqueSet()}. This method runs through all of * the tests in {@link AbstractSetTest}. @@ -698,7 +696,6 @@ public abstract class AbstractMultiSetTest extends AbstractCollectionTest } } - /** * Compare the current serialized form of the MultiSet * against the canonical version in SCM. diff --git a/src/test/java/org/apache/commons/collections4/queue/UnmodifiableQueueTest.java b/src/test/java/org/apache/commons/collections4/queue/UnmodifiableQueueTest.java index 8b213ed43..160e739eb 100644 --- a/src/test/java/org/apache/commons/collections4/queue/UnmodifiableQueueTest.java +++ b/src/test/java/org/apache/commons/collections4/queue/UnmodifiableQueueTest.java @@ -116,7 +116,6 @@ public class UnmodifiableQueueTest extends AbstractQueueTest { assertThrows(UnsupportedOperationException.class, () -> queue.poll()); } - @Override public String getCompatibilityVersion() { return "4"; diff --git a/src/test/java/org/apache/commons/collections4/set/AbstractNavigableSetTest.java b/src/test/java/org/apache/commons/collections4/set/AbstractNavigableSetTest.java index 02352be32..4efb28e00 100644 --- a/src/test/java/org/apache/commons/collections4/set/AbstractNavigableSetTest.java +++ b/src/test/java/org/apache/commons/collections4/set/AbstractNavigableSetTest.java @@ -69,7 +69,6 @@ public abstract class AbstractNavigableSetTest extends AbstractSortedSetTest< return new TreeSet<>(); } - /** * Verification extension, will check the order of elements, * the sets should already be verified equal. diff --git a/src/test/java/org/apache/commons/collections4/set/UnmodifiableNavigableSetTest.java b/src/test/java/org/apache/commons/collections4/set/UnmodifiableNavigableSetTest.java index c62802592..cf090a713 100644 --- a/src/test/java/org/apache/commons/collections4/set/UnmodifiableNavigableSetTest.java +++ b/src/test/java/org/apache/commons/collections4/set/UnmodifiableNavigableSetTest.java @@ -124,7 +124,6 @@ public class UnmodifiableNavigableSetTest extends AbstractNavigableSetTest assertNull(c, "natural order, so comparator should be null"); } - @Override public String getCompatibilityVersion() { return "4.1"; diff --git a/src/test/java/org/apache/commons/collections4/set/UnmodifiableSortedSetTest.java b/src/test/java/org/apache/commons/collections4/set/UnmodifiableSortedSetTest.java index 574cb4292..b91fcf7cf 100644 --- a/src/test/java/org/apache/commons/collections4/set/UnmodifiableSortedSetTest.java +++ b/src/test/java/org/apache/commons/collections4/set/UnmodifiableSortedSetTest.java @@ -121,7 +121,6 @@ public class UnmodifiableSortedSetTest extends AbstractSortedSetTest { assertNull(c, "natural order, so comparator should be null"); } - @Override public String getCompatibilityVersion() { return "4"; diff --git a/src/test/java/org/apache/commons/collections4/trie/PatriciaTrie2Test.java b/src/test/java/org/apache/commons/collections4/trie/PatriciaTrie2Test.java index f4802c5bd..326565df6 100644 --- a/src/test/java/org/apache/commons/collections4/trie/PatriciaTrie2Test.java +++ b/src/test/java/org/apache/commons/collections4/trie/PatriciaTrie2Test.java @@ -40,7 +40,6 @@ public class PatriciaTrie2Test extends AbstractOrderedMapTest { return false; } - @Override public String getCompatibilityVersion() { return "4";