struts-dev mailing list archives

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

----- Original Message ----- 
From: "Paul Sundling" <tomcatlist@tkz.net>
To: "Struts Developers List" <struts-dev@jakarta.apache.org>
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

<fixcrlf
         srcdir="${src.java.dir}"
         tab="remove"
         tablength="4"/>

- Indrajit



---------------------------------------------------------------------
To unsubscribe, e-mail: struts-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: struts-dev-help@jakarta.apache.org


Mime
View raw message