[COLLECTIONS-519] Revert after discussion on the mailinglist.
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/collections/trunk@1591832 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Thomas Neidhart
parent
c48007f44b
commit
d61e967bad
|
|
@ -22,9 +22,6 @@
|
||||||
<body>
|
<body>
|
||||||
|
|
||||||
<release version="4.1" date="TBD" description="">
|
<release version="4.1" date="TBD" description="">
|
||||||
<action issue="COLLECTIONS-519" dev="tn" type="fix" due-to="Radoslav Paskalev, Daniel Feist">
|
|
||||||
Constructors of *Utils classes are now protected to allow sub-classing.
|
|
||||||
</action>
|
|
||||||
<action issue="COLLECTIONS-512" dev="tn" type="fix" due-to="Cyrille Artho">
|
<action issue="COLLECTIONS-512" dev="tn" type="fix" due-to="Cyrille Artho">
|
||||||
"TransformingComparator" did not comply with the contract of Object#equals.
|
"TransformingComparator" did not comply with the contract of Object#equals.
|
||||||
</action>
|
</action>
|
||||||
|
|
|
||||||
|
|
@ -56,7 +56,7 @@ public class ClosureUtils {
|
||||||
/**
|
/**
|
||||||
* This class is not normally instantiated.
|
* This class is not normally instantiated.
|
||||||
*/
|
*/
|
||||||
protected ClosureUtils() {}
|
private ClosureUtils() {}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Gets a Closure that always throws an exception.
|
* Gets a Closure that always throws an exception.
|
||||||
|
|
|
||||||
|
|
@ -185,7 +185,7 @@ public class CollectionUtils {
|
||||||
/**
|
/**
|
||||||
* <code>CollectionUtils</code> should not normally be instantiated.
|
* <code>CollectionUtils</code> should not normally be instantiated.
|
||||||
*/
|
*/
|
||||||
protected CollectionUtils() {}
|
private CollectionUtils() {}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Returns the immutable EMPTY_COLLECTION with generic type safety.
|
* Returns the immutable EMPTY_COLLECTION with generic type safety.
|
||||||
|
|
|
||||||
|
|
@ -43,7 +43,7 @@ public class ComparatorUtils {
|
||||||
/**
|
/**
|
||||||
* ComparatorUtils should not normally be instantiated.
|
* ComparatorUtils should not normally be instantiated.
|
||||||
*/
|
*/
|
||||||
protected ComparatorUtils() {}
|
private ComparatorUtils() {}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Comparator for natural sort order.
|
* Comparator for natural sort order.
|
||||||
|
|
|
||||||
|
|
@ -34,7 +34,7 @@ public class EnumerationUtils {
|
||||||
/**
|
/**
|
||||||
* EnumerationUtils is not normally instantiated.
|
* EnumerationUtils is not normally instantiated.
|
||||||
*/
|
*/
|
||||||
protected EnumerationUtils() {}
|
private EnumerationUtils() {}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Creates a list based on an enumeration.
|
* Creates a list based on an enumeration.
|
||||||
|
|
|
||||||
|
|
@ -41,7 +41,7 @@ public class FactoryUtils {
|
||||||
/**
|
/**
|
||||||
* This class is not normally instantiated.
|
* This class is not normally instantiated.
|
||||||
*/
|
*/
|
||||||
protected FactoryUtils() {}
|
private FactoryUtils() {}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Gets a Factory that always throws an exception.
|
* Gets a Factory that always throws an exception.
|
||||||
|
|
|
||||||
|
|
@ -119,7 +119,7 @@ public class IteratorUtils {
|
||||||
/**
|
/**
|
||||||
* IteratorUtils is not normally instantiated.
|
* IteratorUtils is not normally instantiated.
|
||||||
*/
|
*/
|
||||||
protected IteratorUtils() {}
|
private IteratorUtils() {}
|
||||||
|
|
||||||
// Empty
|
// Empty
|
||||||
//-----------------------------------------------------------------------
|
//-----------------------------------------------------------------------
|
||||||
|
|
|
||||||
|
|
@ -46,7 +46,7 @@ public class ListUtils {
|
||||||
/**
|
/**
|
||||||
* <code>ListUtils</code> should not normally be instantiated.
|
* <code>ListUtils</code> should not normally be instantiated.
|
||||||
*/
|
*/
|
||||||
protected ListUtils() {}
|
private ListUtils() {}
|
||||||
|
|
||||||
//-----------------------------------------------------------------------
|
//-----------------------------------------------------------------------
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -92,7 +92,7 @@ public class MapUtils {
|
||||||
/**
|
/**
|
||||||
* <code>MapUtils</code> should not normally be instantiated.
|
* <code>MapUtils</code> should not normally be instantiated.
|
||||||
*/
|
*/
|
||||||
protected MapUtils() {}
|
private MapUtils() {}
|
||||||
|
|
||||||
// Type safe getters
|
// Type safe getters
|
||||||
//-------------------------------------------------------------------------
|
//-------------------------------------------------------------------------
|
||||||
|
|
|
||||||
|
|
@ -48,7 +48,7 @@ public class MultiMapUtils {
|
||||||
/**
|
/**
|
||||||
* <code>MultiMapUtils</code> should not normally be instantiated.
|
* <code>MultiMapUtils</code> should not normally be instantiated.
|
||||||
*/
|
*/
|
||||||
protected MultiMapUtils() {
|
private MultiMapUtils() {
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
|
||||||
|
|
@ -74,7 +74,7 @@ public class PredicateUtils {
|
||||||
/**
|
/**
|
||||||
* This class is not normally instantiated.
|
* This class is not normally instantiated.
|
||||||
*/
|
*/
|
||||||
protected PredicateUtils() {}
|
private PredicateUtils() {}
|
||||||
|
|
||||||
// Simple predicates
|
// Simple predicates
|
||||||
//-----------------------------------------------------------------------------
|
//-----------------------------------------------------------------------------
|
||||||
|
|
|
||||||
|
|
@ -40,7 +40,7 @@ public class QueueUtils {
|
||||||
/**
|
/**
|
||||||
* <code>QueueUtils</code> should not normally be instantiated.
|
* <code>QueueUtils</code> should not normally be instantiated.
|
||||||
*/
|
*/
|
||||||
protected QueueUtils() {}
|
private QueueUtils() {}
|
||||||
|
|
||||||
//-----------------------------------------------------------------------
|
//-----------------------------------------------------------------------
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -69,7 +69,7 @@ public class SetUtils {
|
||||||
/**
|
/**
|
||||||
* <code>SetUtils</code> should not normally be instantiated.
|
* <code>SetUtils</code> should not normally be instantiated.
|
||||||
*/
|
*/
|
||||||
protected SetUtils() {}
|
private SetUtils() {}
|
||||||
|
|
||||||
//-----------------------------------------------------------------------
|
//-----------------------------------------------------------------------
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -20,11 +20,11 @@ import java.util.Collection;
|
||||||
import java.util.Map;
|
import java.util.Map;
|
||||||
import java.util.Set;
|
import java.util.Set;
|
||||||
|
|
||||||
|
import org.apache.commons.collections4.set.UnmodifiableSet;
|
||||||
import org.apache.commons.collections4.collection.UnmodifiableCollection;
|
import org.apache.commons.collections4.collection.UnmodifiableCollection;
|
||||||
import org.apache.commons.collections4.iterators.UnmodifiableMapIterator;
|
import org.apache.commons.collections4.iterators.UnmodifiableMapIterator;
|
||||||
import org.apache.commons.collections4.map.EntrySetToMapIteratorAdapter;
|
import org.apache.commons.collections4.map.EntrySetToMapIteratorAdapter;
|
||||||
import org.apache.commons.collections4.map.UnmodifiableEntrySet;
|
import org.apache.commons.collections4.map.UnmodifiableEntrySet;
|
||||||
import org.apache.commons.collections4.set.UnmodifiableSet;
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Utilities for working with "split maps:" objects that implement {@link Put}
|
* Utilities for working with "split maps:" objects that implement {@link Put}
|
||||||
|
|
@ -41,7 +41,7 @@ public class SplitMapUtils {
|
||||||
/**
|
/**
|
||||||
* <code>SplitMapUtils</code> should not normally be instantiated.
|
* <code>SplitMapUtils</code> should not normally be instantiated.
|
||||||
*/
|
*/
|
||||||
protected SplitMapUtils() {}
|
private SplitMapUtils() {}
|
||||||
|
|
||||||
//-----------------------------------------------------------------------
|
//-----------------------------------------------------------------------
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -66,7 +66,7 @@ public class TransformerUtils {
|
||||||
/**
|
/**
|
||||||
* This class is not normally instantiated.
|
* This class is not normally instantiated.
|
||||||
*/
|
*/
|
||||||
protected TransformerUtils() {}
|
private TransformerUtils() {}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Gets a transformer that always throws an exception.
|
* Gets a transformer that always throws an exception.
|
||||||
|
|
|
||||||
|
|
@ -29,7 +29,7 @@ public class TrieUtils {
|
||||||
/**
|
/**
|
||||||
* {@link TrieUtils} should not normally be instantiated.
|
* {@link TrieUtils} should not normally be instantiated.
|
||||||
*/
|
*/
|
||||||
protected TrieUtils() {}
|
private TrieUtils() {}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Returns an unmodifiable instance of a {@link Trie}
|
* Returns an unmodifiable instance of a {@link Trie}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue