diff --git a/src/main/java/org/apache/commons/collections4/ArrayUtils.java b/src/main/java/org/apache/commons/collections4/ArrayUtils.java index 61cbb4821d..e997a12f5a 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 @@ */ 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 97e187b71e..3ae1da8737 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 static List collate(final Iterable iterableA, final I 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 054c070a26..f7eb95c5eb 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 int size() { } } - /** * 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 79ed44dccc..1cdb18d37d 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 6d7bcd6221..a9269952a4 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 static Set emptyIfNull(final Set set) { 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 b1bf9bcafe..4e8cceadd6 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 Collection values() { } } - /** * 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 c971469b3e..f9e09f0f7d 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 int hashCode() { 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 3c429dfbc1..dbd319f25b 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 @@ protected SortedBag decorated() { 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 485bc5ec9a..d30e347344 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 int hashCode() { 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 a56e239eb9..ebbc098996 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 @@ protected SortedBag decorated() { 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 3420f4ddfe..91f8822401 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 int hashCode() { } } - @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 3c226ebfdf..9aaeda2a3f 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 @@ protected SortedBag getSortedBag() { 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 7c5bed3e53..755b10eea0 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 int hashCode() { return decorated().hashCode(); } - @Override public int getCount(final Object object) { return getBag().getCount(object); @@ -132,7 +131,6 @@ public boolean remove(final Object object, final int nCopies) { return getBag().remove(object, nCopies); } - @Override public boolean add(final E object, final int nCopies) { return getBag().add(transform(object), nCopies); diff --git a/src/main/java/org/apache/commons/collections4/bag/TransformedSortedBag.java b/src/main/java/org/apache/commons/collections4/bag/TransformedSortedBag.java index cd192db538..8b7d16c123 100644 --- a/src/main/java/org/apache/commons/collections4/bag/TransformedSortedBag.java +++ b/src/main/java/org/apache/commons/collections4/bag/TransformedSortedBag.java @@ -112,7 +112,6 @@ protected SortedBag getSortedBag() { 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 421b49d3a6..c1905a29b8 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 boolean add(final E object) { 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 0c38a97d4d..c5becd8d66 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 @@ protected DualTreeBidiMap createBidiMap(final Map normalMap, final M return new DualTreeBidiMap<>(normalMap, reverseMap, inverseMap); } - @Override public Comparator comparator() { return ((SortedMap) normalMap).comparator(); @@ -205,7 +204,6 @@ public OrderedBidiMap inverseOrderedBidiMap() { 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 c6e46fe5e1..ec02fc8e9b 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 @@ private Node nextSmaller(final Node node, final DataElement dataElem 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 13fc7588d3..b946528413 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 @@ protected void setCollection(final Collection coll) { 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 97544b51c8..065232f263 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 boolean retainAll(final Collection coll) { 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 399221690f..25b69551c0 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 @@ protected Collection decorated() { 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 44ddcbfdbb..609cf9407b 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 @@ protected ListIterator getListIterator() { 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 1d8583ba89..192d668d46 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 @@ protected MapIterator getMapIterator() { 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 5738ac1943..1a8e9f7bf0 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 @@ protected OrderedMapIterator getOrderedMapIterator() { 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 cf1915d13b..1b4b786a37 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 @@ private void init() { } } - @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 7bf975a274..0b2b4e1231 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 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 94811a9b1b..4aa37815bb 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 @@ private void updateCurrentIterator() { } } - /** * 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 ce38757eb7..7a37bd9d25 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 static PeekingIterator peekingIterator(final 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 467a821075..b8b5efdc9c 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 static PushbackIterator pushbackIterator(final Iterator(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 857c52d3ca..b3ace8563c 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 @@ private void init() { } } - @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 1cdd37ad11..09d9fe18bf 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 @@ protected Map.Entry getMapEntry() { 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 2c39bf5c45..514eef6935 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 MultiKey(final K key1, final K key2) { 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 ce1b539a79..83267b6be0 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 @@ protected void init() { header = createHeaderNode(); } - @Override public int size() { return size; @@ -116,7 +115,6 @@ public E get(final int index) { return node.getValue(); } - @Override public Iterator iterator() { return listIterator(); @@ -132,7 +130,6 @@ public ListIterator listIterator(final int fromIndex) { return new LinkedListIterator<>(this, fromIndex); } - @Override public int indexOf(final Object value) { int i = 0; @@ -172,7 +169,6 @@ public boolean containsAll(final Collection coll) { return true; } - @Override public Object[] toArray() { return toArray(new Object[size]); @@ -210,7 +206,6 @@ public List subList(final int fromIndexInclusive, final int toIndexExclusive) return new LinkedSubList<>(this, fromIndexInclusive, toIndexExclusive); } - @Override public boolean add(final E value) { addLast(value); @@ -237,7 +232,6 @@ public boolean addAll(final int index, final Collection coll) { return true; } - @Override public E remove(final int index) { final Node node = getNode(index, false); @@ -279,7 +273,6 @@ public boolean removeAll(final Collection coll) { return modified; } - /** * {@inheritDoc} *

@@ -315,7 +308,6 @@ public void clear() { 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 6c8d469455..23ee150653 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 int hashCode() { 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 fa22991369..7a64c8d9f5 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 int hashCode() { return decorated().hashCode(); } - @Override public E get(final int index) { return decorated().get(index); @@ -126,7 +125,6 @@ public E remove(final int index) { return decorated().remove(index); } - @Override public void add(final int index, final E object) { validate(object); diff --git a/src/main/java/org/apache/commons/collections4/list/TransformedList.java b/src/main/java/org/apache/commons/collections4/list/TransformedList.java index 8631e8eee6..a55c9a2ef3 100644 --- a/src/main/java/org/apache/commons/collections4/list/TransformedList.java +++ b/src/main/java/org/apache/commons/collections4/list/TransformedList.java @@ -124,7 +124,6 @@ public int hashCode() { return decorated().hashCode(); } - @Override public E get(final int index) { return getList().get(index); @@ -145,7 +144,6 @@ public E remove(final int index) { 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 711e3bd046..fc4dbad106 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 V get(final Object key) { 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 2f6bb707e0..a1a273e55e 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 boolean remove(final Object obj) { } - 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 c060985444..08795a850c 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 int hashCode() { 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 668a9db931..c7e55ce270 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 int hashCode() { 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 f522897ca0..0093f6af16 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 int hashCode() { } } - @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 6ffebdaa8d..aa916951f0 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 @@ protected Queue decorated() { 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 f79fc111e2..a96b86c6f0 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 E get(final int index) { 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 b5b031a473..8d8f51f2f0 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 @@ protected Queue decorated() { 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 710a256212..e716fd8ef5 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 @@ protected Queue getQueue() { 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 c8dc509734..219e9d774c 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 boolean retainAll(final Collection coll) { 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 4ada1916e6..83a7f72c8d 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 @@ private Snake getMiddleSnake(final int start1, final int end1, final int start2, 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 92f13d2fd6..0017579f80 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 @@ protected NavigableSet decorated() { 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 d27477216e..c0422264ee 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 @@ protected NavigableSet decorated() { 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 82f0740e8a..b92937c042 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 @@ protected NavigableSet decorated() { 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 62ab39f4fb..af2c7eb9dc 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 @@ TrieEntry followLeft(TrieEntry node) { } } - @Override public Comparator comparator() { return getKeyAnalyzer(); @@ -1387,7 +1386,6 @@ public String toString() { } } - /** * This is an entry set view of the {@link org.apache.commons.collections4.Trie} as returned by {@link Map#entrySet()}. */ @@ -2418,7 +2416,6 @@ public void remove() { } } - /** * Reads the content of the stream. */ 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 6ea3af0f60..70636a8f41 100644 --- a/src/main/java/org/apache/commons/collections4/trie/UnmodifiableTrie.java +++ b/src/main/java/org/apache/commons/collections4/trie/UnmodifiableTrie.java @@ -74,7 +74,6 @@ public UnmodifiableTrie(final Trie trie) { 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 2f8083aed5..b632a57245 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 void testGetDoubleValue() { 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 void testGetByteValue() { 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 6a8cc3cc2f..ec6ab5729a 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 void testUnmodifiableTrie() { } - 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 88f6cbc4e7..e4f66f9463 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 SortedBag makeConfirmedCollection() { return new TreeBag<>(); } - @Override public void resetEmpty() { this.setCollection(CollectionSortedBag.collectionSortedBag(makeObject())); @@ -149,6 +148,5 @@ public SortedBag getCollection() { 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 d88cb81264..2fa2d8634d 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 CollectionBagTest() { super(CollectionBagTest.class.getSimpleName()); } - @Override public Bag makeObject() { return CollectionBag.collectionBag(new HashBag<>()); @@ -91,7 +90,6 @@ protected int getIterationBehaviour() { // 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 e65c2db8d5..5861a9a424 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 CollectionSortedBagTest() { super(CollectionSortedBagTest.class.getSimpleName()); } - /** * Overridden because SortedBags don't allow null elements (normally). * @return false @@ -110,7 +109,6 @@ public T[] getOtherNonNullElements() { return (T[]) elements; } - @Override public String getCompatibilityVersion() { return "4"; @@ -123,7 +121,6 @@ public String getCompatibilityVersion() { // 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 5777ef2ea6..e6f5a60790 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 SynchronizedBagTest() { 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 fd7516d205..bfa9805ef2 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 String[] ignoredTests() { 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 0bd179d008..34cd50d30a 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 AbstractComparatorTest(final String testName) { super(testName); } - /** * Implement this method to return a list of sorted objects. * @@ -204,7 +203,6 @@ public void testComparatorCompatibility() throws IOException, ClassNotFoundExcep } } - // 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 d9217604cb..65a9a5f251 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 final void nullElementsInArrayToGetInstance() { 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 77a11bd898..0248403a39 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.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 5243725c16..45d411db32 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 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 620ec1e3e4..3e9e12fc5f 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 @@ protected void setUp() throws Exception { // 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 ac65d2e510..8d7ac8d195 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 void setUp() { testList.addAll(Arrays.asList(testArray)); } - @Override public boolean supportsRemove() { return false; diff --git a/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java b/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java index dfd1aefb47..762865f5e2 100644 --- a/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java +++ b/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java @@ -829,7 +829,6 @@ private void backwardTest(final ListIterator iter, int i) { "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 5d19ce181e..5f23d87734 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 void testMapContainsValue() { verify(); } - /** * Tests Map.equals(Object) */ @@ -1750,7 +1749,6 @@ public void verify() { } } - /** * Bulk test {@link Map#keySet()}. This method runs through all of * the tests in {@link AbstractSetTest}. @@ -1942,7 +1940,6 @@ protected int getIterationBehaviour(){ // 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 2f74d2652b..f7da0a7dfa 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 void testMapGetWithTransformer() { 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 02f8ecd49f..d3b1292b44 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 331b11e477..a72652bf78 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 MultiSet makeFullCollection() { return multiset; } - @Override public void resetEmpty() { this.setCollection(makeObject()); @@ -615,7 +614,6 @@ public void testMultiSetHashCode() { assertEquals(total, multiset2.hashCode()); } - /** * Bulk test {@link MultiSet#uniqueSet()}. This method runs through all of * the tests in {@link AbstractSetTest}. @@ -698,7 +696,6 @@ protected int getIterationBehaviour() { } } - /** * 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 8b213ed433..160e739eb7 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 void testPoll() { 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 02352be329..4efb28e00e 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 NavigableSet makeConfirmedCollection() { 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 c628025920..cf090a7133 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 void testComparator() { 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 574cb4292d..b91fcf7cf6 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 void testComparator() { 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 f4802c5bdb..326565df6b 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 boolean isAllowNullKey() { return false; } - @Override public String getCompatibilityVersion() { return "4";