[COLLECTIONS-317] AbstractHashedMap(Map map) constructor calls overridable non-final method

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 4.0-alpha1, 4.0
• Component/s: Map
• Assignee: Unassigned
• Reporter: Sebb
• Created: 16/Feb/09 16:18
• Updated: 09/Nov/14 14:31
• Resolved: 28/Feb/13 20:04

Description

The constructor AbstractHashedMap(Map map) calls the public non-final method putAll() from the same class.

This is likely to cause problems if the method is ever overridden, as the parent may not have been completely initialised when the sub-class method is invoked.

Possible solution: extract the code into a common private method. Or make the method final.