Re: [csv] Rename methods on XXXLexer methods on CSVLexer

2012-03-29 Thread Benedikt Ritter
Am 29. März 2012 22:07 schrieb sebb : > On 29 March 2012 20:45, Benedikt Ritter wrote: >> Hey, >> >> how about renaming CSVLexer.simpleTokenLexer() and >> CSVLexer.encapsulatedTokenLexer(), so that the method names express >> what the methods do? >> For example simpleTokenLexer() could be renamed

Re: [csv] Rename methods on XXXLexer methods on CSVLexer

2012-03-29 Thread sebb
On 29 March 2012 20:45, Benedikt Ritter wrote: > Hey, > > how about renaming CSVLexer.simpleTokenLexer() and > CSVLexer.encapsulatedTokenLexer(), so that the method names express > what the methods do? > For example simpleTokenLexer() could be renamed to parseSimpleToken or > parseSimpleTokenConte

[csv] Rename methods on XXXLexer methods on CSVLexer

2012-03-29 Thread Benedikt Ritter
Hey, how about renaming CSVLexer.simpleTokenLexer() and CSVLexer.encapsulatedTokenLexer(), so that the method names express what the methods do? For example simpleTokenLexer() could be renamed to parseSimpleToken or parseSimpleTokenContent. Likewise encapsulatedTokenLexer() could be renamed to par