[ 
https://issues.apache.org/jira/browse/CSV-79?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sebb resolved CSV-79.
---------------------

    Resolution: Fixed

Renamed method to isCommentingEnabled
                
> CSVFormat.isCommentingDisabled() is confusing/confused
> ------------------------------------------------------
>
>                 Key: CSV-79
>                 URL: https://issues.apache.org/jira/browse/CSV-79
>             Project: Commons CSV
>          Issue Type: Bug
>            Reporter: Sebb
>
> The Javadoc for CSVFormat.isCommentingDisabled() says:
> {code}
> /**
>  * Specifies whether comments are supported by this format.
>  * 
>  * @return <tt>true</tt> is comments are supported, <tt>false</tt> otherwise
>  */
> {code}
> however the method actually does the opposite, as the name suggests.
> Now we could just fix the Javadoc, but given that the other isXXX methods 
> return a positive result this would be inconsistent.
> Also, it's generally better to return positive setting.
> So I think renaming the method as "isCommentingEnabled" - and fixing the 
> method code - would be better.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to