You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by tn...@apache.org on 2014/05/02 10:58:41 UTC
svn commit: r1591832 - in /commons/proper/collections/trunk/src: changes/
main/java/org/apache/commons/collections4/
Author: tn
Date: Fri May 2 08:58:40 2014
New Revision: 1591832
URL: http://svn.apache.org/r1591832
Log:
[COLLECTIONS-519] Revert after discussion on the mailinglist.
Modified:
commons/proper/collections/trunk/src/changes/changes.xml
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ClosureUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/CollectionUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ComparatorUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/EnumerationUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/FactoryUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/IteratorUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ListUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/MapUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/MultiMapUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/PredicateUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/QueueUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/SetUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/SplitMapUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/TransformerUtils.java
commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/TrieUtils.java
Modified: commons/proper/collections/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/changes/changes.xml?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/changes/changes.xml (original)
+++ commons/proper/collections/trunk/src/changes/changes.xml Fri May 2 08:58:40 2014
@@ -22,9 +22,6 @@
<body>
<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">
"TransformingComparator" did not comply with the contract of Object#equals.
</action>
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ClosureUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ClosureUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ClosureUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ClosureUtils.java Fri May 2 08:58:40 2014
@@ -56,7 +56,7 @@ public class ClosureUtils {
/**
* This class is not normally instantiated.
*/
- protected ClosureUtils() {}
+ private ClosureUtils() {}
/**
* Gets a Closure that always throws an exception.
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/CollectionUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/CollectionUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/CollectionUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/CollectionUtils.java Fri May 2 08:58:40 2014
@@ -185,7 +185,7 @@ public class CollectionUtils {
/**
* <code>CollectionUtils</code> should not normally be instantiated.
*/
- protected CollectionUtils() {}
+ private CollectionUtils() {}
/**
* Returns the immutable EMPTY_COLLECTION with generic type safety.
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ComparatorUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ComparatorUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ComparatorUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ComparatorUtils.java Fri May 2 08:58:40 2014
@@ -43,7 +43,7 @@ public class ComparatorUtils {
/**
* ComparatorUtils should not normally be instantiated.
*/
- protected ComparatorUtils() {}
+ private ComparatorUtils() {}
/**
* Comparator for natural sort order.
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/EnumerationUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/EnumerationUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/EnumerationUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/EnumerationUtils.java Fri May 2 08:58:40 2014
@@ -34,7 +34,7 @@ public class EnumerationUtils {
/**
* EnumerationUtils is not normally instantiated.
*/
- protected EnumerationUtils() {}
+ private EnumerationUtils() {}
/**
* Creates a list based on an enumeration.
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/FactoryUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/FactoryUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/FactoryUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/FactoryUtils.java Fri May 2 08:58:40 2014
@@ -41,7 +41,7 @@ public class FactoryUtils {
/**
* This class is not normally instantiated.
*/
- protected FactoryUtils() {}
+ private FactoryUtils() {}
/**
* Gets a Factory that always throws an exception.
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/IteratorUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/IteratorUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/IteratorUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/IteratorUtils.java Fri May 2 08:58:40 2014
@@ -119,7 +119,7 @@ public class IteratorUtils {
/**
* IteratorUtils is not normally instantiated.
*/
- protected IteratorUtils() {}
+ private IteratorUtils() {}
// Empty
//-----------------------------------------------------------------------
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ListUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ListUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ListUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/ListUtils.java Fri May 2 08:58:40 2014
@@ -46,7 +46,7 @@ public class ListUtils {
/**
* <code>ListUtils</code> should not normally be instantiated.
*/
- protected ListUtils() {}
+ private ListUtils() {}
//-----------------------------------------------------------------------
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/MapUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/MapUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/MapUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/MapUtils.java Fri May 2 08:58:40 2014
@@ -92,7 +92,7 @@ public class MapUtils {
/**
* <code>MapUtils</code> should not normally be instantiated.
*/
- protected MapUtils() {}
+ private MapUtils() {}
// Type safe getters
//-------------------------------------------------------------------------
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/MultiMapUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/MultiMapUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/MultiMapUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/MultiMapUtils.java Fri May 2 08:58:40 2014
@@ -48,7 +48,7 @@ public class MultiMapUtils {
/**
* <code>MultiMapUtils</code> should not normally be instantiated.
*/
- protected MultiMapUtils() {
+ private MultiMapUtils() {
}
/**
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/PredicateUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/PredicateUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/PredicateUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/PredicateUtils.java Fri May 2 08:58:40 2014
@@ -74,7 +74,7 @@ public class PredicateUtils {
/**
* This class is not normally instantiated.
*/
- protected PredicateUtils() {}
+ private PredicateUtils() {}
// Simple predicates
//-----------------------------------------------------------------------------
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/QueueUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/QueueUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/QueueUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/QueueUtils.java Fri May 2 08:58:40 2014
@@ -40,7 +40,7 @@ public class QueueUtils {
/**
* <code>QueueUtils</code> should not normally be instantiated.
*/
- protected QueueUtils() {}
+ private QueueUtils() {}
//-----------------------------------------------------------------------
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/SetUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/SetUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/SetUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/SetUtils.java Fri May 2 08:58:40 2014
@@ -69,7 +69,7 @@ public class SetUtils {
/**
* <code>SetUtils</code> should not normally be instantiated.
*/
- protected SetUtils() {}
+ private SetUtils() {}
//-----------------------------------------------------------------------
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/SplitMapUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/SplitMapUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/SplitMapUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/SplitMapUtils.java Fri May 2 08:58:40 2014
@@ -20,11 +20,11 @@ import java.util.Collection;
import java.util.Map;
import java.util.Set;
+import org.apache.commons.collections4.set.UnmodifiableSet;
import org.apache.commons.collections4.collection.UnmodifiableCollection;
import org.apache.commons.collections4.iterators.UnmodifiableMapIterator;
import org.apache.commons.collections4.map.EntrySetToMapIteratorAdapter;
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}
@@ -41,7 +41,7 @@ public class SplitMapUtils {
/**
* <code>SplitMapUtils</code> should not normally be instantiated.
*/
- protected SplitMapUtils() {}
+ private SplitMapUtils() {}
//-----------------------------------------------------------------------
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/TransformerUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/TransformerUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/TransformerUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/TransformerUtils.java Fri May 2 08:58:40 2014
@@ -66,7 +66,7 @@ public class TransformerUtils {
/**
* This class is not normally instantiated.
*/
- protected TransformerUtils() {}
+ private TransformerUtils() {}
/**
* Gets a transformer that always throws an exception.
Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/TrieUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/TrieUtils.java?rev=1591832&r1=1591831&r2=1591832&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/TrieUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/TrieUtils.java Fri May 2 08:58:40 2014
@@ -29,7 +29,7 @@ public class TrieUtils {
/**
* {@link TrieUtils} should not normally be instantiated.
*/
- protected TrieUtils() {}
+ private TrieUtils() {}
/**
* Returns an unmodifiable instance of a {@link Trie}