This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-collections.git
commit e74b48b03b275b6d8f6b7d29c9bd4c4c6d14def6 Author: Gary Gregory <[email protected]> AuthorDate: Sun Jan 4 11:04:47 2026 -0500 Remove old comments --- src/main/java/org/apache/commons/collections4/IteratorUtils.java | 1 - src/main/java/org/apache/commons/collections4/SetUtils.java | 1 - .../org/apache/commons/collections4/comparators/ComparatorChain.java | 2 +- .../apache/commons/collections4/comparators/FixedOrderComparator.java | 4 ---- .../apache/commons/collections4/iterators/EnumerationIterator.java | 2 -- src/main/java/org/apache/commons/collections4/map/SingletonMap.java | 3 --- src/test/java/org/apache/commons/collections4/AbstractObjectTest.java | 1 - .../commons/collections4/collection/AbstractCollectionTest.java | 1 - .../commons/collections4/multimap/AbstractMultiValuedMapTest.java | 1 - 9 files changed, 1 insertion(+), 15 deletions(-) diff --git a/src/main/java/org/apache/commons/collections4/IteratorUtils.java b/src/main/java/org/apache/commons/collections4/IteratorUtils.java index cf27afb19..ed65c964f 100644 --- a/src/main/java/org/apache/commons/collections4/IteratorUtils.java +++ b/src/main/java/org/apache/commons/collections4/IteratorUtils.java @@ -392,7 +392,6 @@ public class IteratorUtils { return new IteratorIterable<>(iterator, true); } - // Bounded /** * Decorates the specified iterator to return at most the given number * of elements. diff --git a/src/main/java/org/apache/commons/collections4/SetUtils.java b/src/main/java/org/apache/commons/collections4/SetUtils.java index a6029f9d1..9b77528df 100644 --- a/src/main/java/org/apache/commons/collections4/SetUtils.java +++ b/src/main/java/org/apache/commons/collections4/SetUtils.java @@ -480,7 +480,6 @@ public class SetUtils { return Collections.synchronizedSet(set); } - // SortedSet /** * Returns a synchronized sorted set backed by the given sorted set. * <p> diff --git a/src/main/java/org/apache/commons/collections4/comparators/ComparatorChain.java b/src/main/java/org/apache/commons/collections4/comparators/ComparatorChain.java index 5916d5b74..6e42f2935 100644 --- a/src/main/java/org/apache/commons/collections4/comparators/ComparatorChain.java +++ b/src/main/java/org/apache/commons/collections4/comparators/ComparatorChain.java @@ -60,7 +60,7 @@ public class ComparatorChain<E> implements Comparator<E>, Serializable { private final List<Comparator<E>> comparatorChain; /** Order - false (clear) = ascend; true (set) = descend. */ private final BitSet orderingBits; - /** Whether the chain has been "locked". */ + /** Whether the chain has been "locked". */ private boolean isLocked; /** diff --git a/src/main/java/org/apache/commons/collections4/comparators/FixedOrderComparator.java b/src/main/java/org/apache/commons/collections4/comparators/FixedOrderComparator.java index 42490a70f..fa9b3a25a 100644 --- a/src/main/java/org/apache/commons/collections4/comparators/FixedOrderComparator.java +++ b/src/main/java/org/apache/commons/collections4/comparators/FixedOrderComparator.java @@ -90,7 +90,6 @@ public class FixedOrderComparator<T> implements Comparator<T>, Serializable { /** The behavior in the case of an unknown object */ private UnknownObjectBehavior unknownObjectBehavior = UnknownObjectBehavior.EXCEPTION; - // Constructors /** * Constructs an empty FixedOrderComparator. */ @@ -127,7 +126,6 @@ public class FixedOrderComparator<T> implements Comparator<T>, Serializable { } } - // Methods for adding items /** * Adds an item, which compares as after all items known to the Comparator. * If the item is already known to the Comparator, its old position is @@ -178,7 +176,6 @@ public class FixedOrderComparator<T> implements Comparator<T>, Serializable { } } - // Comparator methods /** * Compares two objects according to the order of this Comparator. * <p> @@ -245,7 +242,6 @@ public class FixedOrderComparator<T> implements Comparator<T>, Serializable { return Objects.hash(counter, isLocked, map, unknownObjectBehavior); } - // Bean methods / state querying methods /** * Returns true if modifications cannot be made to the FixedOrderComparator. * FixedOrderComparators cannot be modified once they have performed a comparison. diff --git a/src/main/java/org/apache/commons/collections4/iterators/EnumerationIterator.java b/src/main/java/org/apache/commons/collections4/iterators/EnumerationIterator.java index 95333b407..4030569e7 100644 --- a/src/main/java/org/apache/commons/collections4/iterators/EnumerationIterator.java +++ b/src/main/java/org/apache/commons/collections4/iterators/EnumerationIterator.java @@ -36,7 +36,6 @@ public class EnumerationIterator<E> implements Iterator<E> { /** The last object retrieved */ private E last; - // Constructors /** * Constructs a new {@code EnumerationIterator} that will not * function until {@link #setEnumeration(Enumeration)} is called. @@ -77,7 +76,6 @@ public class EnumerationIterator<E> implements Iterator<E> { return enumeration; } - // Iterator interface /** * Returns true if the underlying enumeration has more elements. * diff --git a/src/main/java/org/apache/commons/collections4/map/SingletonMap.java b/src/main/java/org/apache/commons/collections4/map/SingletonMap.java index b70367ca0..c8a29a8ae 100644 --- a/src/main/java/org/apache/commons/collections4/map/SingletonMap.java +++ b/src/main/java/org/apache/commons/collections4/map/SingletonMap.java @@ -336,7 +336,6 @@ public class SingletonMap<K, V> return getKey(); } - // Map /** * Gets the value mapped to the key specified. * @@ -351,7 +350,6 @@ public class SingletonMap<K, V> return null; } - // KeyValue /** * Gets the key. * @@ -413,7 +411,6 @@ public class SingletonMap<K, V> return Objects.equals(value, getValue()); } - // BoundedMap /** * Is the map currently full, always true. * diff --git a/src/test/java/org/apache/commons/collections4/AbstractObjectTest.java b/src/test/java/org/apache/commons/collections4/AbstractObjectTest.java index db2eaa5cb..4a31e6e84 100644 --- a/src/test/java/org/apache/commons/collections4/AbstractObjectTest.java +++ b/src/test/java/org/apache/commons/collections4/AbstractObjectTest.java @@ -74,7 +74,6 @@ public abstract class AbstractObjectTest extends BulkTest { return retval.toString(); } - // protected implementation /** * Gets the version of Collections that this object tries to * maintain serialization compatibility with. Defaults to 4, due to diff --git a/src/test/java/org/apache/commons/collections4/collection/AbstractCollectionTest.java b/src/test/java/org/apache/commons/collections4/collection/AbstractCollectionTest.java index 425b83b00..1645d72c1 100644 --- a/src/test/java/org/apache/commons/collections4/collection/AbstractCollectionTest.java +++ b/src/test/java/org/apache/commons/collections4/collection/AbstractCollectionTest.java @@ -578,7 +578,6 @@ public abstract class AbstractCollectionTest<E> extends AbstractObjectTest { this.confirmed = confirmed; } - // Tests /** * Tests {@link Collection#add(Object)}. */ 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 ff6a7b152..f6ec78ce0 100644 --- a/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java +++ b/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java @@ -435,7 +435,6 @@ public abstract class AbstractMultiValuedMapTest<K, V> extends AbstractObjectTes return new MultiValuedMapAsMapTest(); } - // Bulk Tests /** * Bulk test {@link MultiValuedMap#entries()}. This method runs through all * of the tests in {@link AbstractCollectionTest}. After modification
