You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by se...@apache.org on 2010/10/04 10:38:51 UTC

svn commit: r1004142 - /directory/buildtools/trunk/checkstyle/src/main/resources/directory-checks.xml

Author: seelmann
Date: Mon Oct  4 08:38:50 2010
New Revision: 1004142

URL: http://svn.apache.org/viewvc?rev=1004142&view=rev
Log:
No Javadoc required for private fields; Allow protected fields

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=1004142&r1=1004141&r2=1004142&view=diff
==============================================================================
--- directory/buildtools/trunk/checkstyle/src/main/resources/directory-checks.xml (original)
+++ directory/buildtools/trunk/checkstyle/src/main/resources/directory-checks.xml Mon Oct  4 08:38:50 2010
@@ -85,12 +85,16 @@
         <!-- Checks for Javadoc comments.                     -->
         <!-- See http://checkstyle.sf.net/config_javadoc.html -->
         <module name="JavadocMethod">
+          <!-- No Javadoc required for private methods -->
           <property name="scope" value="protected"/>
         </module>
         <module name="JavadocType">
           <property name="scope" value="public"/>
         </module>
-        <module name="JavadocVariable"/>
+        <module name="JavadocVariable">
+          <!-- No Javadoc required for private fields -->
+          <property name="scope" value="protected"/>
+        </module>
         <module name="JavadocStyle">
             <property name="checkFirstSentence" value="false"/>
         </module>
@@ -193,7 +197,11 @@
         <module name="FinalClass"/>
         <module name="HideUtilityClassConstructor"/>
         <module name="InterfaceIsType"/>
-        <module name="VisibilityModifier"/>
+        <module name="VisibilityModifier">
+            <property name="packageAllowed" value="true"/>
+            <property name="protectedAllowed" value="true"/>
+        </module>
+        
 
 
         <!-- Miscellaneous other checks.                   -->