On 17 March 2012 01:39,  <ebo...@apache.org> wrote:
> Author: ebourg
> Date: Sat Mar 17 01:39:04 2012
> New Revision: 1301852
>
> URL: http://svn.apache.org/viewvc?rev=1301852&view=rev
> Log:
> Header support (CSV-65)
>
> Added:
>    
> 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/CSVFormat.java
>    
> commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVParser.java
>    commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/package.html
>    
> commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVFormatTest.java
>    
> commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVParserTest.java
>    
> commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVPrinterTest.java

<snip/>

> Added: 
> 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=1301852&view=auto
> ==============================================================================
> --- 
> commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVRecord.java 
> (added)
> +++ 
> commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVRecord.java 
> Sat Mar 17 01:39:04 2012
> @@ -0,0 +1,87 @@
<snip/>

> + *
> + * @author Emmanuel Bourg

The ASF deprecates @author comments in code; these should go in the
pom / release notes / changes instead.

> + */
> +public class CSVRecord implements Serializable, Iterable<String> {
> +

Does the class need to be public?

Does it need to be Serializable?

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

Reply via email to