You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by fe...@apache.org on 2010/09/06 21:32:00 UTC
svn commit: r993131 -
/directory/buildtools/trunk/checkstyle/src/main/resources/directory-checks.xml
Author: felixk
Date: Mon Sep 6 19:32:00 2010
New Revision: 993131
URL: http://svn.apache.org/viewvc?rev=993131&view=rev
Log:
Checks that a local variable or a parameter does not shadow a field that is defined in the same class:
- ignore constructor parameters
- ignore the parameter of a property setter method, where the property setter method for field "xyz" has name "setXyz", one parameter named "xyz", and return type void
Modified:
directory/buildtools/trunk/checkstyle/src/main/resources/directory-checks.xml
Modified: directory/buildtools/trunk/checkstyle/src/main/resources/directory-checks.xml
URL: http://svn.apache.org/viewvc/directory/buildtools/trunk/checkstyle/src/main/resources/directory-checks.xml?rev=993131&r1=993130&r2=993131&view=diff
==============================================================================
--- directory/buildtools/trunk/checkstyle/src/main/resources/directory-checks.xml (original)
+++ directory/buildtools/trunk/checkstyle/src/main/resources/directory-checks.xml Mon Sep 6 19:32:00 2010
@@ -169,7 +169,10 @@
<module name="DoubleCheckedLocking"/> <!-- MY FAVOURITE -->
<module name="EmptyStatement"/>
<module name="EqualsHashCode"/>
- <module name="HiddenField"/>
+ <module name="HiddenField">
+ <property name="ignoreSetter" value="true"/>
+ <property name="ignoreConstructorParameter" value="true"/>
+ </module>
<module name="IllegalInstantiation"/>
<module name="InnerAssignment"/>
<!-- module name="MagicNumber"/ -->