Re: commons-csv git commit: [CSV-201] Do not use RuntimeException in CSVParser.iterator().new Iterator() {...}.getNextRecord(). Use an IllegalStateException instead (KISS for now, no need for a custom

2016-10-28 Thread Gilles
On Fri, 28 Oct 2016 08:03:58 -0700, Gary Gregory wrote: I do not think reverting is right since the previous is obviously bogus, to me, that is. Throwing a RE is never useful, again IMO. We should just evolve the code from here. Would you like to outline choices? 1) NSEE 2) Custom 3) ISE 4) ?

[GitHub] commons-collections pull request #19: COLLECTIONS-575: Add synchronized queu...

2016-10-28 Thread Xaerxess
GitHub user Xaerxess opened a pull request: https://github.com/apache/commons-collections/pull/19 COLLECTIONS-575: Add synchronized queue wrapper Added `QueueUtils#synchronizedQueue(Queue)` wrapper and `SynchronizedQueue` with tests. Please check if I used proper conventions as

Re: [complex] commons-complex module

2016-10-28 Thread Rob Tompkins
> On Oct 25, 2016, at 5:59 PM, Gary Gregory wrote: > > I am against spinning out org.apache.commons.math4.complex out into yet > another component. The argument that Gilles refuses to > support org.apache.commons.math4.complex because he does not want to > support all of

Re: commons-csv git commit: [CSV-201] Do not use RuntimeException in CSVParser.iterator().new Iterator() {...}.getNextRecord(). Use an IllegalStateException instead (KISS for now, no need for a custom

2016-10-28 Thread Gary Gregory
I do not think reverting is right since the previous is obviously bogus, to me, that is. Throwing a RE is never useful, again IMO. We should just evolve the code from here. Would you like to outline choices? 1) NSEE 2) Custom 3) ISE 4) ? Gary (will be offline most of today) On Oct 28, 2016

Re: commons-csv git commit: [CSV-201] Do not use RuntimeException in CSVParser.iterator().new Iterator() {...}.getNextRecord(). Use an IllegalStateException instead (KISS for now, no need for a custom

2016-10-28 Thread Benedikt Ritter
Hello, Benedikt Ritter schrieb am Mi., 26. Okt. 2016 um 21:38 Uhr: > Hello, > > schrieb am Di., 25. Okt. 2016 um 22:59 Uhr: > > Repository: commons-csv > Updated Branches: > refs/heads/master b2a50d4a3 -> 1c7a9910b > > > [CSV-201] Do not use