Author: luc
Date: Sun Aug  9 10:07:05 2009
New Revision: 802509

URL: http://svn.apache.org/viewvc?rev=802509&view=rev
Log:
prevent checkstyle from complaining about missing hidden constructor in 
LazyHolder classes
these classes are private internal classes and completely under control, there 
is no
risk anyone instantiate them and in fact it would not harm either.
Adding an hidden constructor to make checkstyle happy would make cobertura 
unhappy and displaying
very low coverage since cobertura currently (August 2009) cannot mark 
untestable methods, so it
seems better to use checkstyle filtering feature in these specific and 
controlled cases

Modified:
    commons/sandbox/nabla/trunk/checkstyle.xml

Modified: commons/sandbox/nabla/trunk/checkstyle.xml
URL: 
http://svn.apache.org/viewvc/commons/sandbox/nabla/trunk/checkstyle.xml?rev=802509&r1=802508&r2=802509&view=diff
==============================================================================
--- commons/sandbox/nabla/trunk/checkstyle.xml (original)
+++ commons/sandbox/nabla/trunk/checkstyle.xml Sun Aug  9 10:07:05 2009
@@ -83,6 +83,19 @@
                              PLUS_ASSIGN, QUESTION, RCURLY, SL, SLIST, 
SL_ASSIGN, SR,
                              SR_ASSIGN, STAR, STAR_ASSIGN"/>
         </module>
-    </module>
-    <module name="NewlineAtEndOfFile"/>
+
+        <!-- Required for SuppressionCommentFilter below -->
+        <module name="FileContentsHolder"/>
+
+      </module>
+  
+      <!-- Setup special comments to suppress specific checks from source 
files -->
+      <module name="SuppressionCommentFilter">
+          <property name="offCommentFormat" value="CHECKSTYLE\: stop 
HideUtilityClassConstructor"/>
+          <property name="onCommentFormat"  value="CHECKSTYLE\: resume 
HideUtilityClassConstructor"/>
+          <property name="checkFormat"      
value="HideUtilityClassConstructor"/>
+      </module>
+
+      <module name="NewlineAtEndOfFile"/>
+
 </module>


Reply via email to