Re: svn commit: r1509450 - in /commons/proper/csv/trunk/src: main/java/org/apache/commons/csv/CSVRecord.java test/java/org/apache/commons/csv/CSVRecordIntTest.java

2013-08-01 Thread Emmanuel Bourg
Le 01/08/2013 23:57, ggreg...@apache.org a écrit : Add CSVRecord#getInt(String) API and tests. All of these methods are already implemented in [configuration] with pretty much all the types you might need (Enums, URLs, colors, emails, dates, calendars, primitives, arrays, etc). If CSVRecord

Re: svn commit: r1509450 - in /commons/proper/csv/trunk/src: main/java/org/apache/commons/csv/CSVRecord.java test/java/org/apache/commons/csv/CSVRecordIntTest.java

2013-08-01 Thread sebb
On 1 August 2013 22:57, ggreg...@apache.org wrote: Author: ggregory Date: Thu Aug 1 21:57:46 2013 New Revision: 1509450 URL: http://svn.apache.org/r1509450 Log: Add CSVRecord#getInt(String) API and tests. -1 I think this is out of scope for CSV. Added:

Re: svn commit: r1509450 - in /commons/proper/csv/trunk/src: main/java/org/apache/commons/csv/CSVRecord.java test/java/org/apache/commons/csv/CSVRecordIntTest.java

2013-08-01 Thread James Carman
You are -1ing a code modification, which means this counts as a veto. Are you sure you want to veto this? On Thu, Aug 1, 2013 at 7:28 PM, sebb seb...@gmail.com wrote: On 1 August 2013 22:57, ggreg...@apache.org wrote: Author: ggregory Date: Thu Aug 1 21:57:46 2013 New Revision: 1509450

Re: svn commit: r1509450 - in /commons/proper/csv/trunk/src: main/java/org/apache/commons/csv/CSVRecord.java test/java/org/apache/commons/csv/CSVRecordIntTest.java

2013-08-01 Thread sebb
On 2 August 2013 00:44, James Carman ja...@carmanconsulting.com wrote: You are -1ing a code modification, which means this counts as a veto. Are you sure you want to veto this? Yes. We don't have agreement that CSV should include these methods. On Thu, Aug 1, 2013 at 7:28 PM, sebb

Re: svn commit: r1509450 - in /commons/proper/csv/trunk/src: main/java/org/apache/commons/csv/CSVRecord.java test/java/org/apache/commons/csv/CSVRecordIntTest.java

2013-08-01 Thread James Carman
How about we put it to a vote? Would you withdraw your veto if a majority votes against it? On Thu, Aug 1, 2013 at 7:55 PM, sebb seb...@gmail.com wrote: On 2 August 2013 00:44, James Carman ja...@carmanconsulting.com wrote: You are -1ing a code modification, which means this counts as a veto.