Re: Checkstyle, Reloaded

2012-02-06 Thread Pascal Sancho
Hi, rather than display capabilities, we should consider how human can interpret a long line. It is established that he can read and understand 10 entities (words or signs) at a glance. Coding usages give a useful line width about 75-85 characters. We should add to this an average indent.

Re: Checkstyle, Reloaded

2012-02-06 Thread Chris Bowditch
On 03/02/2012 21:20, Glenn Adams wrote: Hi Glen, which version of checkstyle are you using? there are two errors in parsing the proposed checkstyle file with 5.1; Vincent says checkstyle v5.5 was used in his original e-mail. !--property name=ignoreEnhancedForColon value=false/ --

Re: Checkstyle, Reloaded

2012-02-06 Thread Chris Bowditch
On 03/02/2012 17:45, Vincent Hennebert wrote: Hi All, Hi Vincent, it is well-known that people are not happy with the Checkstyle file we have in FOP. And there’s no point enforcing the application of Checkstyle rules if we don’t agree with them in the first place. Agreed. I’ve finally

Re: Checkstyle, Reloaded

2012-02-06 Thread Glenn Adams
overall, the i believe the issue of whitespace usage is a matter of personal style, and should not be subject to strict rule enforcement; as long as i can use CSOFF to disable rules on source files i create, then i can accept rules which encode different usage patterns; i believe it is more

Re: Checkstyle, Reloaded

2012-02-06 Thread Alexios Giotis
Hi, I can't see a point having checkstyle rules and then adding CSOFF on new files to disable them. It is faster to read, debug or fix source code when there is uniformity rather than every file having the personal style of the initial author. It would be helpful to additionally have