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?
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