Author: skitching
Date: Mon Mar  6 21:11:35 2006
New Revision: 383784

URL: http://svn.apache.org/viewcvs?rev=383784&view=rev
Log:
Enable checkstyle report for logging code.

Added:
    jakarta/commons/proper/logging/contrib/simon/jcl2/checkstyle.xml
Modified:
    jakarta/commons/proper/logging/contrib/simon/jcl2/pom.xml

Added: jakarta/commons/proper/logging/contrib/simon/jcl2/checkstyle.xml
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/logging/contrib/simon/jcl2/checkstyle.xml?rev=383784&view=auto
==============================================================================
--- jakarta/commons/proper/logging/contrib/simon/jcl2/checkstyle.xml (added)
+++ jakarta/commons/proper/logging/contrib/simon/jcl2/checkstyle.xml Mon Mar  6 
21:11:35 2006
@@ -0,0 +1,62 @@
+<?xml version="1.0"?>
+
+<!DOCTYPE module PUBLIC
+    "-//Puppy Crawl//DTD Check Configuration 1.1//EN"
+    "http://www.puppycrawl.com/dtds/configuration_1_1.dtd";>
+
+<!-- commons digester customization of default Checkstyle behavior -->
+<module name="Checker">
+  <property name="localeLanguage" value="en"/>
+
+  <!-- require a package.html for each package -->
+  <module name="PackageHtml"/>
+
+  <module name="TreeWalker">
+    <!-- Verify that EVERY source file has the appropriate license
+    <module name="Header">
+      <property name="headerFile" value="file-header.txt"/>
+      <property name="ignoreLines" value="1,3"/>
+    </module>
+     -->
+
+    <!-- no tabs allowed in files -->
+    <module name="TabCharacter"/>
+    
+    <!-- forbid lines over 100 chars in length -->
+    <module name="LineLength">
+      <property name="max" value="100"/>
+    </module>
+
+    <!-- check sane import statements -->
+    <module name="AvoidStarImport"/>
+    <module name="RedundantImport"/>
+    <module name="UnusedImports"/>
+    <module name="IllegalImport"/>
+
+       <!-- javadoc checks -->
+    <module name="JavadocType"/>
+
+       <!-- code layout checks -->    
+    <module name="EmptyBlock"/>
+    <module name="LeftCurly"/>
+    <module name="RightCurly"/>
+    <module name="NeedBraces"/>
+
+       <!-- code logic checks -->    
+    <module name="CovariantEquals"/>
+    <module name="FinalLocalVariable"/>
+    <module name="MagicNumber"/>
+    <module name="MissingSwitchDefault"/>
+    <module name="ModifiedControlVariable"/>
+    <module name="SimplifyBooleanExpression"/>
+    <module name="SimplifyBooleanReturn"/>
+    <module name="StringLiteralEquality"/>
+    <module name="SuperClone"/>
+    <module name="SuperFinalize"/>
+    <module name="JUnitTestCase"/>
+    <module name="ParameterAssignment"/>
+    <module name="MultipleVariableDeclarations"/>
+    <module name="UnnecessaryParentheses"/>
+  </module>
+</module>
+

Modified: jakarta/commons/proper/logging/contrib/simon/jcl2/pom.xml
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/logging/contrib/simon/jcl2/pom.xml?rev=383784&r1=383783&r2=383784&view=diff
==============================================================================
--- jakarta/commons/proper/logging/contrib/simon/jcl2/pom.xml (original)
+++ jakarta/commons/proper/logging/contrib/simon/jcl2/pom.xml Mon Mar  6 
21:11:35 2006
@@ -53,15 +53,13 @@
 
   <reporting>
     <plugins>
-    <!--
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-checkstyle-plugin</artifactId>
         <configuration>
-          <configLocation>config/avalon_checks.xml</configLocation>
+          <configLocation>checkstyle.xml</configLocation>
         </configuration>
       </plugin>
-      -->
     </plugins>
   </reporting>
 </project>



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to