commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From t.@apache.org
Subject svn commit: r1683013 - in /commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4: ./ multimap/
Date Mon, 01 Jun 2015 22:27:23 GMT
Author: tn
Date: Mon Jun  1 22:27:22 2015
New Revision: 1683013

URL: http://svn.apache.org/r1683013
Log:
First review of MultiValuedMap: formatting, throw NullPointerException for null inputs.

Modified:
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/MultiMapUtils.java
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractListValuedMap.java
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapDecorator.java
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractSetValuedMap.java
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/MultiValuedHashMap.java
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMap.java
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/package-info.java

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=1683013&r1=1683012&r2=1683013&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
Mon Jun  1 22:27:22 2015
@@ -30,10 +30,8 @@ import org.apache.commons.collections4.m
 /**
  * Provides utility methods and decorators for {@link MultiValuedMap} instances.
  * <p>
- * It contains various type safe and null safe methods.
- * <p>
- * It also provides the following decorators:
- *
+ * It contains various type safe and null safe methods. Additionally, it provides
+ * the following decorators:
  * <ul>
  *   <li>{@link #unmodifiableMultiValuedMap(MultiValuedMap)}</li>
  *   <li>{@link #transformedMultiValuedMap(MultiValuedMap, Transformer, Transformer)}</li>
@@ -47,8 +45,7 @@ public class MultiMapUtils {
     /**
      * <code>MultiMapUtils</code> should not normally be instantiated.
      */
-    private MultiMapUtils() {
-    }
+    private MultiMapUtils() {}
 
     /**
      * An empty {@link UnmodifiableMultiValuedMap}.
@@ -60,8 +57,8 @@ public class MultiMapUtils {
     /**
      * Returns immutable EMPTY_MULTI_VALUED_MAP with generic type safety.
      *
-     * @param <K> the type of key in the map
-     * @param <V> the type of value in the map
+     * @param <K>  the type of key in the map
+     * @param <V>  the type of value in the map
      * @return immutable and empty <code>MultiValuedMap</code>
      */
     @SuppressWarnings("unchecked")
@@ -75,10 +72,10 @@ public class MultiMapUtils {
      * Returns an immutable empty <code>MultiValuedMap</code> if the argument
is
      * <code>null</code>, or the argument itself otherwise.
      *
-     * @param <K> the type of key in the map
-     * @param <V> the type of value in the map
-     * @param map the map, possibly <code>null</code>
-     * @return an empty <code>MultiValuedMap</code> if the argument is <code>null</code>
+     * @param <K>  the type of key in the map
+     * @param <V>  the type of value in the map
+     * @param map  the map, may be null
+     * @return an empty {@link MultiValuedMap} if the argument is null
      */
     @SuppressWarnings("unchecked")
     public static <K, V> MultiValuedMap<K, V> emptyIfNull(final MultiValuedMap<K,
V> map) {
@@ -88,10 +85,10 @@ public class MultiMapUtils {
     /**
      * Null-safe check if the specified <code>MultiValuedMap</code> is empty.
      * <p>
-     * Null returns true.
+     * If the provided map is null, returns true.
      *
-     * @param map the map to check, may be null
-     * @return true if empty or null
+     * @param map  the map to check, may be null
+     * @return true if the map is empty or null
      */
     public static boolean isEmpty(final MultiValuedMap<?, ?> map) {
         return map == null || map.isEmpty();
@@ -103,11 +100,11 @@ public class MultiMapUtils {
     /**
      * Gets a Collection from <code>MultiValuedMap</code> in a null-safe manner.
      *
-     * @param <K> the key type
-     * @param <V> the value type
-     * @param map the <code>MultiValuedMap</code> to use
-     * @param key the key to look up
-     * @return the Collection in the <code>MultiValuedMap</code>, <code>null</code>
if map input is null
+     * @param <K>  the key type
+     * @param <V>  the value type
+     * @param map  the {@link MultiValuedMap} to use
+     * @param key  the key to look up
+     * @return the Collection in the {@link MultiValuedMap}, or null if input map is null
      */
     public static <K, V> Collection<V> getCollection(final MultiValuedMap<K,
V> map, final K key) {
         if (map != null) {
@@ -116,17 +113,19 @@ public class MultiMapUtils {
         return null;
     }
 
+    // TODO: review the getValuesAsXXX methods - depending on the actual MultiValuedMap type,
changes
+    // to the returned collection might update the backing map. This should be clarified
and/or prevented.
+
     /**
      * Gets a List from <code>MultiValuedMap</code> in a null-safe manner.
      *
-     * @param <K> the key type
-     * @param <V> the value type
-     * @param map the <code>MultiValuedMap</code> to use
-     * @param key the key to look up
-     * @return the Collection in the <code>MultiValuedMap</code> as List,
-     *         <code>null</code> if map input is null
+     * @param <K>  the key type
+     * @param <V>  the value type
+     * @param map  the {@link MultiValuedMap} to use
+     * @param key  the key to look up
+     * @return the Collection in the {@link MultiValuedMap} as List, or null if input map
is null
      */
-    public static <K, V> List<V> getList(MultiValuedMap<K, V> map, K key)
{
+    public static <K, V> List<V> getValuesAsList(final MultiValuedMap<K, V>
map, final K key) {
         if (map != null) {
             Collection<V> col = map.get(key);
             if (col instanceof List) {
@@ -140,14 +139,13 @@ public class MultiMapUtils {
     /**
      * Gets a Set from <code>MultiValuedMap</code> in a null-safe manner.
      *
-     * @param <K> the key type
-     * @param <V> the value type
-     * @param map the <code>MultiValuedMap</code> to use
-     * @param key the key to look up
-     * @return the Collection in the <code>MultiValuedMap</code> as Set,
-     *         <code>null</code> if map input is null
+     * @param <K>  the key type
+     * @param <V>  the value type
+     * @param map  the {@link MultiValuedMap} to use
+     * @param key  the key to look up
+     * @return the Collection in the {@link MultiValuedMap} as Set, or null if input map
is null
      */
-    public static <K, V> Set<V> getSet(MultiValuedMap<K, V> map, K key)
{
+    public static <K, V> Set<V> getValuesAsSet(final MultiValuedMap<K, V>
map, final K key) {
         if (map != null) {
             Collection<V> col = map.get(key);
             if (col instanceof Set) {
@@ -161,14 +159,13 @@ public class MultiMapUtils {
     /**
      * Gets a Bag from <code>MultiValuedMap</code> in a null-safe manner.
      *
-     * @param <K> the key type
-     * @param <V> the value type
-     * @param map the <code>MultiValuedMap</code> to use
-     * @param key the key to look up
-     * @return the Collection in the <code>MultiValuedMap</code> as Bag,
-     *         <code>null</code> if map input is null
+     * @param <K>  the key type
+     * @param <V>  the value type
+     * @param map  the {@link MultiValuedMap} to use
+     * @param key  the key to look up
+     * @return the Collection in the {@link MultiValuedMap} as Bag, or null if input map
is null
      */
-    public static <K, V> Bag<V> getBag(MultiValuedMap<K, V> map, K key)
{
+    public static <K, V> Bag<V> getValuesAsBag(final MultiValuedMap<K, V>
map, final K key) {
         if (map != null) {
             Collection<V> col = map.get(key);
             if (col instanceof Bag) {
@@ -183,13 +180,14 @@ public class MultiMapUtils {
     // -----------------------------------------------------------------------
 
     /**
-     * Creates a {@link ListValuedMap} with a {@link java.util.HashMap HashMap} as its internal
storage.
+     * Creates a {@link ListValuedMap} with an {@link java.util.ArrayList ArrayList} as
+     * collection class to store the values mapped to a key.
      *
-     * @param <K> the key type
-     * @param <V> the value type
+     * @param <K>  the key type
+     * @param <V>  the value type
      * @return a new <code>ListValuedMap</code>
      */
-    public static <K, V> ListValuedMap<K, V> createListValuedHashMap() {
+    public static <K, V> ListValuedMap<K, V> newListValuedHashMap() {
         return MultiValuedHashMap.<K, V>listValuedHashMap();
     }
 
@@ -197,25 +195,25 @@ public class MultiMapUtils {
      * Creates a {@link ListValuedMap} with a {@link java.util.HashMap HashMap} as its internal
      * storage which maps the keys to list of type <code>listClass</code>.
      *
-     * @param <K> the key type
-     * @param <V> the value type
-     * @param <C> the List class type
+     * @param <K>  the key type
+     * @param <V>  the value type
+     * @param <C>  the List class type
      * @param listClass the class of the list
-     * @return a new <code>ListValuedMap</code>
+     * @return a new {@link ListValuedMap}
      */
-    public static <K, V, C extends List<V>> ListValuedMap<K, V> createListValuedHashMap(final
Class<C> listClass) {
+    public static <K, V, C extends List<V>> ListValuedMap<K, V> newListValuedHashMap(final
Class<C> listClass) {
         return MultiValuedHashMap.<K, V, C>listValuedHashMap(listClass);
     }
 
     /**
-     * Creates a {@link SetValuedMap} with a {@link java.util.HashMap HashMap} as its internal
-     * storage
+     * Creates a {@link SetValuedMap} with an {@link java.util.HashSet HashSet} as
+     * collection class to store the values mapped to a key.
      *
-     * @param <K> the key type
-     * @param <V> the value type
-     * @return a new <code>SetValuedMap</code>
+     * @param <K>  the key type
+     * @param <V>  the value type
+     * @return a new {@link SetValuedMap}
      */
-    public static <K, V> SetValuedMap<K, V> createSetValuedHashMap() {
+    public static <K, V> SetValuedMap<K, V> newSetValuedHashMap() {
         return MultiValuedHashMap.<K, V>setValuedHashMap();
     }
 
@@ -223,13 +221,13 @@ public class MultiMapUtils {
      * Creates a {@link SetValuedMap} with a {@link java.util.HashMap HashMap} as its internal
      * storage which maps the keys to a set of type <code>setClass</code>
      *
-     * @param <K> the key type
-     * @param <V> the value type
-     * @param <C> the Set class type
-     * @param setClass the class of the set
-     * @return a new <code>SetValuedMap</code>
+     * @param <K>  the key type
+     * @param <V>  the value type
+     * @param <C>  the Set class type
+     * @param setClass  the class of the set
+     * @return a new {@link SetValuedMap}
      */
-    public static <K, V, C extends Set<V>> SetValuedMap<K, V> createSetValuedHashMap(final
Class<C> setClass) {
+    public static <K, V, C extends Set<V>> SetValuedMap<K, V> newSetValuedHashMap(final
Class<C> setClass) {
         return MultiValuedHashMap.<K, V, C>setValuedHashMap(setClass);
     }
 
@@ -240,13 +238,11 @@ public class MultiMapUtils {
      * Returns an <code>UnmodifiableMultiValuedMap</code> backed by the given
      * map.
      *
-     * @param <K> the key type
-     * @param <V> the value type
-     * @param map the <code>MultiValuedMap</code> to make unmodifiable, must
not
-     *        be null
-     * @return an <code>UnmodifiableMultiValuedMap</code> backed by the given
-     *         map
-     * @throws IllegalArgumentException if the map is null
+     * @param <K>  the key type
+     * @param <V>  the value type
+     * @param map  the {@link MultiValuedMap} to decorate, must not be null
+     * @return an unmodifiable {@link MultiValuedMap} backed by the provided map
+     * @throws IllegalArgumentException if map is null
      */
     public static <K, V> MultiValuedMap<K, V> unmodifiableMultiValuedMap(
             final MultiValuedMap<? extends K, ? extends V> map) {
@@ -268,17 +264,13 @@ public class MultiMapUtils {
      * If there are any elements already in the map being decorated, they are
      * NOT transformed.
      *
-     * @param <K> the key type
-     * @param <V> the value type
-     * @param map the <code>MultiValuedMap</code> to transform, must not be
-     *        null, typically empty
-     * @param keyTransformer the transformer for the map keys, null means no
-     *        transformation
-     * @param valueTransformer the transformer for the map values, null means no
-     *        transformation
+     * @param <K>  the key type
+     * @param <V>  the value type
+     * @param map the {@link MultiValuedMap} to transform, must not be null, typically empty
+     * @param keyTransformer the transformer for the map keys, null means no transformation
+     * @param valueTransformer the transformer for the map values, null means no transformation
      * @return a transformed <code>MultiValuedMap</code> backed by the given
map
-     * @throws IllegalArgumentException if the <code>MultiValuedMap</code> is
-     *         null
+     * @throws IllegalArgumentException if map is null
      */
     public static <K, V> MultiValuedMap<K, V> transformedMultiValuedMap(final
MultiValuedMap<K, V> map,
             final Transformer<? super K, ? extends K> keyTransformer,

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractListValuedMap.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractListValuedMap.java?rev=1683013&r1=1683012&r2=1683013&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractListValuedMap.java
(original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractListValuedMap.java
Mon Jun  1 22:27:22 2015
@@ -49,9 +49,9 @@ public abstract class AbstractListValued
      * @param <C> the list type
      * @param map the map to wrap, must not be null
      * @param listClazz the collection class
-     * @throws IllegalArgumentException if the map is null
+     * @throws NullPointerException if the map is null
      */
-    protected <C extends List<V>> AbstractListValuedMap(Map<K, ? super C>
map, Class<C> listClazz) {
+    protected <C extends List<V>> AbstractListValuedMap(final Map<K, ? super
C> map, Class<C> listClazz) {
         super(map, listClazz);
     }
 
@@ -62,11 +62,11 @@ public abstract class AbstractListValued
      * @param map the map to wrap, must not be null
      * @param listClazz the collection class
      * @param initialListCapacity the initial size of the values list
-     * @throws IllegalArgumentException if the map is null or if
-     *         initialListCapacity is negative
+     * @throws NullPointerException if the map is null
+     * @throws IllegalArgumentException if initialListCapacity is negative
      */
-    protected <C extends List<V>> AbstractListValuedMap(Map<K, ? super C>
map, Class<C> listClazz,
-            int initialListCapacity) {
+    protected <C extends List<V>> AbstractListValuedMap(final Map<K, ? super
C> map, Class<C> listClazz,
+                                                        final int initialListCapacity) {
         super(map, listClazz, initialListCapacity);
     }
 
@@ -75,8 +75,7 @@ public abstract class AbstractListValued
      * return an empty list in case the mapping is not present
      *
      * @param key the key to retrieve
-     * @return the <code>List</code> of values, will return an empty
-     *         <code>List</code> for no mapping
+     * @return the <code>List</code> of values, will return an empty {@link List}
for no mapping
      * @throws ClassCastException if the key is of an invalid type
      */
     @Override

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java?rev=1683013&r1=1683012&r2=1683013&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java
(original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java
Mon Jun  1 22:27:22 2015
@@ -77,13 +77,13 @@ public class AbstractMultiValuedMap<K, V
      * @param <C>  the collection type
      * @param map  the map to wrap, must not be null
      * @param collectionClazz  the collection class
-     * @throws IllegalArgumentException if the map is null
+     * @throws NullPointerException if the map is null
      */
     @SuppressWarnings("unchecked")
     protected <C extends Collection<V>> AbstractMultiValuedMap(final Map<K,
? super C> map,
                                                                final Class<C> collectionClazz)
{
         if (map == null) {
-            throw new IllegalArgumentException("Map must not be null");
+            throw new NullPointerException("Map must not be null.");
         }
         this.map = (Map<K, Collection<V>>) map;
         this.collectionFactory = new InstantiateFactory<C>(collectionClazz);
@@ -96,14 +96,14 @@ public class AbstractMultiValuedMap<K, V
      * @param map the map to wrap, must not be null
      * @param collectionClazz the collection class
      * @param initialCollectionCapacity the initial capacity of the collection
-     * @throws IllegalArgumentException if the map is null or if
-     *         initialCollectionCapacity is negative
+     * @throws NullPointerException if the map is null
+     * @throws IllegalArgumentException if initialCollectionCapacity is negative
      */
     @SuppressWarnings("unchecked")
     protected <C extends Collection<V>> AbstractMultiValuedMap(final Map<K,
? super C> map,
             final Class<C> collectionClazz, final int initialCollectionCapacity) {
         if (map == null) {
-            throw new IllegalArgumentException("Map must not be null");
+            throw new NullPointerException("Map must not be null.");
         }
         if (initialCollectionCapacity < 0) {
             throw new IllegalArgumentException("Illegal Capacity: " + initialCollectionCapacity);

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapDecorator.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapDecorator.java?rev=1683013&r1=1683012&r2=1683013&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapDecorator.java
(original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapDecorator.java
Mon Jun  1 22:27:22 2015
@@ -29,8 +29,9 @@ import org.apache.commons.collections4.M
 /**
  * Decorates another <code>MultiValuedMap</code> to provide additional behaviour.
  * <p>
- * Each method call made on this <code>MultiValuedMap</code> is forwarded to
the decorated <code>MultiValuedMap</code>.
- * This class is used as a framework to build to extensions such as synchronized and unmodifiable
behaviour.
+ * Each method call made on this <code>MultiValuedMap</code> is forwarded to
the
+ * decorated <code>MultiValuedMap</code>. This class is used as a framework to
build
+ * to extensions such as synchronized and unmodifiable behaviour.
  *
  * @param <K> the type of key elements
  * @param <V> the type of value elements
@@ -51,12 +52,11 @@ public class AbstractMultiValuedMapDecor
      * Constructor that wraps (not copies).
      *
      * @param map the map to decorate, must not be null
-     * @throws IllegalArgumentException if the map is null
+     * @throws NullPointerException if the map is null
      */
     protected AbstractMultiValuedMapDecorator(final MultiValuedMap<K, V> map) {
         if (map == null) {
-            throw new IllegalArgumentException(
-                    "MultiValuedMap must not be null");
+            throw new NullPointerException("MultiValuedMap must not be null.");
         }
         this.map = map;
     }

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractSetValuedMap.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractSetValuedMap.java?rev=1683013&r1=1683012&r2=1683013&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractSetValuedMap.java
(original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/AbstractSetValuedMap.java
Mon Jun  1 22:27:22 2015
@@ -47,7 +47,7 @@ public abstract class AbstractSetValuedM
      * @param <C> the set type
      * @param map the map to wrap, must not be null
      * @param setClazz the collection class
-     * @throws IllegalArgumentException if the map is null
+     * @throws NullPointerException if the map is null
      */
     protected <C extends Set<V>> AbstractSetValuedMap(Map<K, ? super C>
map, Class<C> setClazz) {
         super(map, setClazz);
@@ -60,8 +60,8 @@ public abstract class AbstractSetValuedM
      * @param map the map to wrap, must not be null
      * @param setClazz the collection class
      * @param initialSetCapacity the initial size of the values set
-     * @throws IllegalArgumentException if the map is null or if
-     *         initialSetCapacity is negative
+     * @throws NullPointerException if the map is null
+     * @throws IllegalArgumentException if initialSetCapacity is negative
      */
     protected <C extends Set<V>> AbstractSetValuedMap(Map<K, ? super C>
map, Class<C> setClazz,
             int initialSetCapacity) {

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/MultiValuedHashMap.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/MultiValuedHashMap.java?rev=1683013&r1=1683012&r2=1683013&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/MultiValuedHashMap.java
(original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/MultiValuedHashMap.java
Mon Jun  1 22:27:22 2015
@@ -163,8 +163,7 @@ public class MultiValuedHashMap<K, V> ex
      *
      * @param initialCapacity the initial capacity of the underlying hash map
      * @param loadFactor the load factor of the underlying hash map
-     * @param initialCollectionCapacity the initial capacity of the Collection
-     *        of values
+     * @param initialCollectionCapacity the initial capacity of the Collection of values
      */
     @SuppressWarnings("unchecked")
     public MultiValuedHashMap(int initialCapacity, float loadFactor, int initialCollectionCapacity)
{
@@ -211,17 +210,13 @@ public class MultiValuedHashMap<K, V> ex
 
     /**
      * Creates a MultiValuedHashMap which creates the value collections using
-     * the supplied <code>collectionClazz</code> and the initial collection
-     * capacity .
+     * the supplied <code>collectionClazz</code> and the initial collection capacity.
      *
-     * @param initialCapacity the initial capacity of the underlying
-     *        <code>HashMap</code>
-     * @param loadFactor the load factor of the underlying <code>HashMap</code>
-     * @param initialCollectionCapacity the initial capacity of the
-     *        <code>Collection</code>
      * @param <C> the collection type
-     * @param collectionClazz the class of the <code>Collection</code> to use
to
-     *        create the value collections
+     * @param initialCapacity the initial capacity of the underlying <code>HashMap</code>
+     * @param loadFactor the load factor of the underlying <code>HashMap</code>
+     * @param initialCollectionCapacity the initial capacity of the <code>Collection</code>
+     * @param collectionClazz the class of the <code>Collection</code> to use
to create the value collections
      */
     protected <C extends Collection<V>> MultiValuedHashMap(int initialCapacity,
float loadFactor,
             final Class<C> collectionClazz, int initialCollectionCapacity) {

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMap.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMap.java?rev=1683013&r1=1683012&r2=1683013&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMap.java
(original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/UnmodifiableMultiValuedMap.java
Mon Jun  1 22:27:22 2015
@@ -60,8 +60,8 @@ public class UnmodifiableMultiValuedMap<
      * @throws IllegalArgumentException if map is null
      */
     @SuppressWarnings("unchecked")
-    public static <K, V> UnmodifiableMultiValuedMap<K, V>
-            unmodifiableMultiValuedMap(MultiValuedMap<? extends K, ? extends V> map)
{
+    public static <K, V> UnmodifiableMultiValuedMap<K, V> unmodifiableMultiValuedMap(
+            MultiValuedMap<? extends K, ? extends V> map) {
         if (map instanceof Unmodifiable) {
             return (UnmodifiableMultiValuedMap<K, V>) map;
         }

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/package-info.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/package-info.java?rev=1683013&r1=1683012&r2=1683013&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/package-info.java
(original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/multimap/package-info.java
Mon Jun  1 22:27:22 2015
@@ -21,7 +21,6 @@
  * The following implementations are provided in the package:
  * <ul>
  *   <li>MultiValuedHashMap - implementation that uses a HashMap to store the data
- *   <li>MultiValuedLinkedHashMap - implementation that uses a LinkedHashMap as backing
map
  * </ul>
  * <p>
  * The following decorators are provided in the package:



Mime
View raw message