[COLLECTIONS-316] LRUMap constructor javadoc and behavior are inconsistent (maxSize cannot be less than 0)

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.1
• Fix Version/s: 4.0-alpha1, 4.0
• Component/s: Map

Environment

n/a

• Assignee: Unassigned
• Reporter: ori
• Created: 10/Feb/09 21:55
• Updated: 09/Nov/14 14:31
• Resolved: 28/Feb/13 20:04

Description

The LRUMap constructor specifies "@param maxSize the maximum size of the map, -1 for no limit"

The first line is then:

if (maxSize < 1)

{ throw new IllegalArgumentException("LRUMap max size must be greater than 0"); }

There is nothing wrong with allowing a negative maxSize. This way the map can be used to order elements relative to access time. I think the above code should be remove and the javadoc should stay the same.