[GUMP@vmgump]: Project commons-vfs2-test (in module apache-commons) failed

2012-10-13 Thread Gump
To whom it may engage... This is an automated request, but not an unsolicited one. For more information please visit http://gump.apache.org/nagged.html, and/or contact the folk at gene...@gump.apache.org. Project commons-vfs2-test has an issue affecting its community integration. This i

[GUMP@vmgump]: Project commons-scxml-test (in module apache-commons) failed

2012-10-13 Thread Gump
To whom it may engage... This is an automated request, but not an unsolicited one. For more information please visit http://gump.apache.org/nagged.html, and/or contact the folk at gene...@gump.apache.org. Project commons-scxml-test has an issue affecting its community integration. This

Re: svn commit: r1397783 - in /commons/proper/csv/trunk/src: main/java/org/apache/commons/csv/CSVFormat.java main/java/org/apache/commons/csv/Lexer.java test/java/org/apache/commons/csv/CSVFormatTest.

2012-10-13 Thread sebb
On 13 October 2012 07:27, wrote: > Author: ggregory > Date: Sat Oct 13 06:27:52 2012 > New Revision: 1397783 > > URL: http://svn.apache.org/viewvc?rev=1397783&view=rev > Log: > Remove DISABLED character hack. -1 (for now) Are you sure this change does not affect performance? The Lexer code now

[CSV] Lexer and Character conversion

2012-10-13 Thread sebb
Before r1397883, the Lexer operated only on char fields; it's now been converted to use Character, which means that unboxing is needed. Also, the Character fields need to be checked for null before use. It has just occurred to me that there is a genuine illegal char value for everything except th

Re: svn commit: r1397783 - in /commons/proper/csv/trunk/src: main/java/org/apache/commons/csv/CSVFormat.java main/java/org/apache/commons/csv/Lexer.java test/java/org/apache/commons/csv/CSVFormatTest.

2012-10-13 Thread Gary Gregory
Thank you for the feedback Sebb. I'll do another pass later today. Gary On Oct 13, 2012, at 6:28, sebb wrote: > On 13 October 2012 07:27, wrote: >> Author: ggregory >> Date: Sat Oct 13 06:27:52 2012 >> New Revision: 1397783 >> >> URL: http://svn.apache.org/viewvc?rev=1397783&view=rev >> Log:

Re: svn commit: r1397783 - in /commons/proper/csv/trunk/src: main/java/org/apache/commons/csv/CSVFormat.java main/java/org/apache/commons/csv/Lexer.java test/java/org/apache/commons/csv/CSVFormatTest.

2012-10-13 Thread sebb
On 13 October 2012 13:14, Gary Gregory wrote: > Thank you for the feedback Sebb. I'll do another pass later today. See also my subsequent posting on the dev list. I think that might resolve the performance issue without needing to revert all the changes to CSVFormat. > Gary > > On Oct 13, 2012,

Re: svn commit: r1397783 - in /commons/proper/csv/trunk/src: main/java/org/apache/commons/csv/CSVFormat.java main/java/org/apache/commons/csv/Lexer.java test/java/org/apache/commons/csv/CSVFormatTest.

2012-10-13 Thread Gary Gregory
On Oct 13, 2012, at 8:37, sebb wrote: > On 13 October 2012 13:14, Gary Gregory wrote: >> Thank you for the feedback Sebb. I'll do another pass later today. > > See also my subsequent posting on the dev list. > I think that might resolve the performance issue without needing to > revert all the c

Re: [CSV] Lexer and Character conversion

2012-10-13 Thread sebb
On 13 October 2012 12:55, sebb wrote: > Before r1397883, the Lexer operated only on char fields; it's now been > converted to use Character, which means that unboxing is needed. > > Also, the Character fields need to be checked for null before use. > > It has just occurred to me that there is a ge

Re: [CSV] Lexer and Character conversion

2012-10-13 Thread sebb
On 13 October 2012 14:31, sebb wrote: > On 13 October 2012 12:55, sebb wrote: >> Before r1397883, the Lexer operated only on char fields; it's now been >> converted to use Character, which means that unboxing is needed. >> >> Also, the Character fields need to be checked for null before use. >> >

Re: svn commit: r1397859 - in /commons/proper/csv/trunk: pom.xml src/test/java/org/apache/commons/csv/perf/PerformanceTest.java

2012-10-13 Thread Gary Gregory
On Sat, Oct 13, 2012 at 11:30 AM, wrote: > Author: sebb > Date: Sat Oct 13 15:30:07 2012 > New Revision: 1397859 > > URL: http://svn.apache.org/viewvc?rev=1397859&view=rev > Log: > Drop @Ignore from perf/PerformanceTest; instead add to pom surefire > excludes > This makes it possible to run the t

[continuum] BUILD FAILURE: Apache Commons - Commons CSV -

2012-10-13 Thread Continuum@vmbuild
Group (shared) Maven 2 Build Definition (Java 1.5) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Continuum-Build-Host: vmbuild X-Continuum-Project-Id: 68 X-Continuum-Project-Name: Commons CSV Online report : http://vmbuild.apache.org/cont

Re: RSS?

2012-10-13 Thread Paul Libbrecht
To produce RSS, I've used Jelly quite happily. I know it is not actively supported and has some flaws... but to output xml, it plays really well. Paul Le 13 oct. 2012 à 07:12, Gary Gregory a écrit : > It seems like Rome development has staled. Is the library fully baked? > > I am going to chu

Re: RSS?

2012-10-13 Thread Gary Gregory
I'm looking to consume though. Gary On Oct 13, 2012, at 15:46, Paul Libbrecht wrote: > To produce RSS, I've used Jelly quite happily. > I know it is not actively supported and has some flaws... but to output xml, > it plays really well. > > Paul > > > Le 13 oct. 2012 à 07:12, Gary Gregory a éc

Trouble submitting a patch

2012-10-13 Thread Gary Lucas
Damjan, I just submitted a new Tracker Item, #93, and a patch. Unfortunately, I don't seem to be able to find a control for granting a license. I'm going to Google a bit to see if I can figure it out. But I wanted to let you know in case you saw it before I was able to figure it out. Gary