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"/ -->