On Wed, Jul 2, 2008 at 7:28 PM, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote: > Hi All, > > In the new checkstyle rules file I enabled checks for tab characters, as the > myfaces convention is (AFAIK) to use 4 spaces, not tabs. However the > checkstyle report points out a lot of files containing tabs. > > It's no big deal, but do we want to: > (a) disable the checkstyle rule and ignore tabs or > (b) fix them?
+1 for (b) -M > > Tabs are a minor nuisance when viewing the source as some tools render 4 > spaces, some 8. > > I've written a simple shellscript that can clean this up very easily, and am > happy to do so. The script also removes trailing whitespace from lines, of > which we also appear to have quite a lot. > > But doing this will create some large commit messages and make comparing > files with past versions noisier. It can also cause svn conflicts if people > have modified files they have not yet committed, unless they run the cleanup > script against their own working dir before doing svn update. > > So, option (a) or (b)? > > Regards, Simon > > -- Matthias Wessendorf further stuff: blog: http://matthiaswessendorf.wordpress.com/ sessions: http://www.slideshare.net/mwessendorf mail: matzew-at-apache-dot-org
