[CSV-223] Inconsistency between Javadoc of CSVFormat DEFAULT EXCEL

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.6
• Component/s: Documentation
• External issue URL: https://github.com/apache/commons-csv/pull/26
• External issue ID: 26
• Assignee: Gary D. Gregory
• Reporter: Samuel Martin
• Created: 31/Mar/18 19:56
• Updated: 04/Apr/18 13:38
• Resolved: 03/Apr/18 19:46

Description

According to the Javadoc of CSVFormat 

• CSVFormat.DEFAULT includes among its settings withIgnoreEmptyLines(true)
• CSVFormat.EXCEL includes among its settings withIgnoreEmptyLines(false)
• Then, the same documentation of CSVFormat.EXCEL reads that

  Note: this is currently like RFC4180 plus withAllowMissingColumnNames(true).

One of these three must be wrong (both ignore empty lines, or neither ignores empty lines, or there is another difference that needs to be mentioned).
Edit: after checking the source code, I conclude that the latest sentence is incomplete, and should also mention withIgnoreEmptyLines(false) as a difference with DEFAULT