[COLLECTIONS-390] Inconsistent Javadoc comment and code for mapTransformer(Map<? super I, ? extends O>) in org.apache.commons.collections.TransformerUtils

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Affects Version/s: 4.0
• Component/s: Buffer

Environment

Platform Independent

• Assignee: Unassigned
• Reporter: SHIN HWEI TAN
• Created: 08/Jan/12 08:31
• Updated: 24/Jun/13 19:34
• Resolved: 08/Jan/12 08:32

Description

The Javadoc comment below states that the method "throws IllegalArgumentException if the map is null":
/**
....

• @param map the map to use to transform the objects
• @return the transformer
• @throws IllegalArgumentException if the map is null
  */
  public static <I, O> Transformer<I, O> mapTransformer(Map<? super I, ? extends O> map) { return MapTransformer.mapTransformer(map); }

  However, the method returns a NULL_INSTANCE object instead of throwing IllegalArgumentException when called with null.

Suggested Fixes:
1. Change "@throws IllegalArgumentException if the map is null" and "@return" to "@return NULL_INSTANCE if the map is null".
or
2. Remove the entire "throws IllegalArgumentException if the map is null".