AW: cvs commit: ant/src/main/org/apache/tools/ant DemuxOutputStre am.java

[Ja...@rzf.fin-nrw.de]

> > I know I'm nitpicking here, and I apologize for it, but aren't you
> > creating lines over 80 chars with this reformatting Steve? 
> I picked just
> > one example here. Is this an IDE configuration issue? --DD
> 
> 
> you arent nitpicking; 80 lines is the ant rule; 100 lines is another 
> project.


Mmh checkstyle-config [1] sais 
    <module name="LineLength">
      <property name="max" value="100"/>
      <property name="ignorePattern" value="^ *\* *[^ ]+$"/>
    </module>
sounds like java-length-max is 100, javadoc-length-max isnt set.


Jan


[1]
http://cvs.apache.org/viewcvs.cgi/ant/src/etc/checkstyle/checkstyle-config?v
iew=markup

Re: AW: cvs commit: ant/src/main/org/apache/tools/ant DemuxOutputStre am.java

[Conor MacNeill <co...@apache.org>]

Jan.Materne@rzf.fin-nrw.de wrote:
> 
> Mmh checkstyle-config [1] sais 
>     <module name="LineLength">
>       <property name="max" value="100"/>
>       <property name="ignorePattern" value="^ *\* *[^ ]+$"/>
>     </module>
> sounds like java-length-max is 100, javadoc-length-max isnt set.
> 
> 

I think I set that originally. I probably chose that as a more 
achievable goal to start with.

Conor

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