Author: dennisl
Date: Sat Feb  9 07:15:26 2008
New Revision: 620137

URL: http://svn.apache.org/viewvc?rev=620137&view=rev
Log:
o Adapt maven_checks.xml to how we currently code:
  - Allow empty blocks that contains only text, like a comment.
  - Allow protected member variables.

Modified:
    
maven/plugins/trunk/maven-checkstyle-plugin/src/main/resources/config/maven_checks.xml

Modified: 
maven/plugins/trunk/maven-checkstyle-plugin/src/main/resources/config/maven_checks.xml
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/main/resources/config/maven_checks.xml?rev=620137&r1=620136&r2=620137&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-checkstyle-plugin/src/main/resources/config/maven_checks.xml
 (original)
+++ 
maven/plugins/trunk/maven-checkstyle-plugin/src/main/resources/config/maven_checks.xml
 Sat Feb  9 07:15:26 2008
@@ -131,7 +131,9 @@
         <!-- Checks for blocks. You know, those {}'s         -->
         <!-- See http://checkstyle.sf.net/config_blocks.html -->
         <module name="AvoidNestedBlocks"/>
-        <module name="EmptyBlock"/>
+        <module name="EmptyBlock">
+          <property name="option" value="text"/>
+        </module>
         <module name="NeedBraces"/>
 
 
@@ -160,8 +162,9 @@
         <!-- module name="FinalClass"/ -->
         <!-- module name="HideUtilityClassConstructor"/ -->
         <module name="InterfaceIsType"/>
-        <module name="VisibilityModifier"/>
-
+        <module name="VisibilityModifier">
+          <property name="protectedAllowed" value="true"/>
+        </module>
 
         <!-- Miscellaneous other checks.                   -->
         <!-- See http://checkstyle.sf.net/config_misc.html -->


Reply via email to