You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2013/06/24 16:22:57 UTC

svn commit: r1496064 - /commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVRecord.java

Author: sebb
Date: Mon Jun 24 14:22:56 2013
New Revision: 1496064

URL: http://svn.apache.org/r1496064
Log:
Clarify (hopefully) the Javadoc

Modified:
    commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVRecord.java

Modified: commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVRecord.java
URL: http://svn.apache.org/viewvc/commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVRecord.java?rev=1496064&r1=1496063&r2=1496064&view=diff
==============================================================================
--- commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVRecord.java (original)
+++ commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVRecord.java Mon Jun 24 14:22:56 2013
@@ -104,11 +104,11 @@ public class CSVRecord implements Serial
     }
 
     /**
-     * Checks whether a given column is mapped.
+     * Checks whether a given column is mapped, i.e. its name has been defined to the parser.
      *
      * @param name
      *            the name of the column to be retrieved.
-     * @return whether a given columns is mapped.
+     * @return whether a given column is mapped.
      */
     public boolean isMapped(final String name) {
         return mapping != null ? mapping.containsKey(name) : false;



Re: svn commit: r1496064 - /commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVRecord.java

Posted by Emmanuel Bourg <eb...@apache.org>.
Le 24/06/2013 16:22, sebb@apache.org a écrit :

> -     * Checks whether a given column is mapped.
> +     * Checks whether a given column is mapped, i.e. its name has been defined to the parser.

I'd rather say something like "if the name has been defined in the
header". I would also refer to the 'column name' instead of the 'column'.


Emmanuel Bourg


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org