Niclas Hedhman
Mon, 25 Jan 2010 03:50:23 -0800
On Fri, Jan 22, 2010 at 6:45 AM, Johan Lindquist <jo...@kawoo.co.uk> wrote:
> Seems that the checkstyle version used by the maven checkstyle plugin > has changed and the current 'style' configuration in use is no longer > valid - anyone else seen issues with it? For example, it is complaining > that it is 'Unable to instantiate PackageHtml'. You also made some changes... 1. You made "Trailing Spaces" allowed. 2. You made "TabCharacter" allowed I don't have too much opinion about the first (although I think it is good taste to ensure no trailing spaces), but the second is just nasty, and I would like to have that revoked. What is the reason you think that Tab characteer is anywhere near useful? My objection is that ALL source files where tab character is allowed will get its formatting buggered up, since the use of the character is not well-defined. Some editors have a "every 8th position" and some editors are "every indent" which are drastically different, and incompatible between each other. See example; Open a JDK class in IDEA with indent == 4 (which is the indent JDK uses). Cheers -- Niclas Hedhman, Software Developer http://www.qi4j.org - New Energy for Java I live here; http://tinyurl.com/2qq9er I work here; http://tinyurl.com/2ymelc I relax here; http://tinyurl.com/2cgsug _______________________________________________ general mailing list general@lists.ops4j.org http://lists.ops4j.org/mailman/listinfo/general