[CSV-34] CSVFormat describes itself as immutable, but it is not - in particular it is not thread-safe

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.0
• Assignee: Unassigned
• Reporter: Sebb
• Created: 06/Mar/12 20:41
• Updated: 14/Aug/14 20:06
• Resolved: 09/Mar/12 16:45

Description

CSVFormat describes itself as immutable, but it is not @Immutable - the class fields are all mutable.

The methods that change the fields do so by creating a clone, and returning the changed clone.
So in a sense the class is immutable.

However, the normal expectation is that @Immutable classes are @ThreadSafe.
CSVFormat is not thread-safe, because the fields are not volatile, and the fields are not written & read using a common lock.

The comment needs to be clarified or removed.