struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Indrajit Raychaudhuri" <>
Subject Re: maven checkstyle report
Date Sun, 11 Jan 2004 05:45:30 GMT

----- Original Message ----- 
From: "Paul Sundling" <>
To: "Struts Developers List" <>
Sent: Sunday, 11 January, 2004 04:27
Subject: Re: maven checkstyle report

> Actually, I was realizing that Option 2 is only a quick fix, not a
> solution and that eventually there will still be a bunch of tab errors.
> Realizing that we'll be accepting code from people who may or may not be
> using tabs, it's likely to be a persistant error...  Some assistance
> could be provided by an ant target like this that I used to package my
> own code for a client who hated tabs:
>     <target name="notabs"  description="Replace tab with 4 spaces" >
>         <replaceregexp match="\t"
>             replace="    "
>             flags="g" >
>             <fileset dir="web" excludes="**/images/*" />
>         </replaceregexp>
>     </target>
> Even that assumes a tab is always the same number of spaces, which is
> the whole issue with tabs in the first place.

You could also use the Ant task <fixcrlf/>
For example (assuming Struts convention of '4 whitespaces per tab'), we
could have


- Indrajit

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message