Author: Alex.Kolonitsky
Date: 2009-12-03 10:35:12 -0500 (Thu, 03 Dec 2009)
New Revision: 16054
Added:
root/checkstyle/src/main/resources/exadel-richfaces-checkstyle/richfaces-checkstyle.xml
Removed:
root/checkstyle/src/main/resources/exadel-richfaces-checkstyle/exadel+sun-checkstyle.xml
Log:
RFPL-195 Code style policy
Rename checkstyle file
Deleted:
root/checkstyle/src/main/resources/exadel-richfaces-checkstyle/exadel+sun-checkstyle.xml
===================================================================
---
root/checkstyle/src/main/resources/exadel-richfaces-checkstyle/exadel+sun-checkstyle.xml 2009-12-03
15:09:52 UTC (rev 16053)
+++
root/checkstyle/src/main/resources/exadel-richfaces-checkstyle/exadel+sun-checkstyle.xml 2009-12-03
15:35:12 UTC (rev 16054)
@@ -1,316 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-
-<!DOCTYPE module PUBLIC "-//Puppy Crawl//DTD Check Configuration 1.2//EN"
- "http://www.puppycrawl.com/dtds/configuration_1_2.dtd">
-<module name="Checker">
- <property name="severity" value="error"/>
- <module name="Translation"/>
- <!--<module name="FileLength">-->
- <!--<property name="max" value="1000"/>-->
- <!--<property name="severity"
value="warning"/>-->
- <!--</module>-->
-
- <module name="TreeWalker">
- <property name="severity" value="error"/>
-
- <!-- Checks for Javadoc comments. -->
- <!-- See
http://checkstyle.sourceforge.net/config_javadoc.html -->
- <module name="JavadocMethod">
- <property name="scope" value="public"/>
- <property name="severity" value="warning"/>
- </module>
- <module name="JavadocType">
- <property name="severity" value="warning"/>
- </module>
- <module name="JavadocVariable">
- <property name="scope" value="public"/>
- <property name="severity" value="warning"/>
- </module>
- <module name="JavadocStyle">
- <property name="checkEmptyJavadoc" value="true"/>
- <property name="checkFirstSentence"
value="false"/>
- <property name="severity" value="warning"/>
- </module>
- </module>
-
- <module name="TreeWalker">
- <!-- Checks for Naming Conventions. -->
- <!-- See
http://checkstyle.sourceforge.net/config_naming.html -->
- <module name="ConstantName">
- <property name="severity" value="error"/>
- </module>
- <module name="LocalFinalVariableName">
- <property name="severity" value="error"/>
- </module>
- <module name="LocalVariableName">
- <property name="severity" value="error"/>
- </module>
- <module name="MemberName">
- <property name="severity" value="error"/>
- </module>
- <module name="MethodName">
- <property name="severity" value="error"/>
- </module>
- <module name="PackageName">
- <property name="severity" value="error"/>
- </module>
- <module name="ParameterName">
- <property name="severity" value="error"/>
- </module>
- <module name="StaticVariableName">
- <property name="severity" value="error"/>
- </module>
- <module name="TypeName">
- <property name="severity" value="error"/>
- </module>
- </module>
-
- <module name="TreeWalker">
- <!-- Checks for imports -->
- <!-- See
http://checkstyle.sourceforge.net/config_import.html -->
- <module name="AvoidStarImport">
- <property name="severity" value="warning"/>
- </module>
- <module name="IllegalImport">
- <property name="severity" value="error"/>
- </module>
- <module name="RedundantImport">
- <property name="severity" value="error"/>
- </module>
- <module name="UnusedImports">
- <property name="severity" value="error"/>
- </module>
- </module>
-
- <module name="TreeWalker">
- <!-- Checks for Size Violations. -->
- <!-- See
http://checkstyle.sourceforge.net/config_sizes.html -->
- <module name="LineLength">
- <property name="max" value="120"/>
- <property name="severity" value="warning"/>
- </module>
- <module name="MethodLength">
- <property name="tokens" value="METHOD_DEF"/>
- <property name="max" value="60"/>
- <!-- I think must be 30 -->
- <property name="countEmpty" value="false"/>
- <property name="severity" value="warning"/>
- </module>
- <module name="AnonInnerLength">
- <property name="max" value="40"/>
- <property name="severity" value="error"/>
- </module>
- <module name="ParameterNumber">
- <property name="severity" value="warning"/>
- </module>
- </module>
-
- <module name="TreeWalker">
- <!-- Checks for whitespace -->
- <!-- See
http://checkstyle.sourceforge.net/config_whitespace.html -->
- <!--<module name="TabCharacter"/>-->
- <module name="EmptyForIteratorPad">
- <property name="severity" value="info"/>
- </module>
- <module name="MethodParamPad">
- <property name="severity" value="info"/>
- </module>
- <module name="NoWhitespaceAfter">
- <property name="severity" value="info"/>
- </module>
- <module name="NoWhitespaceBefore">
- <property name="severity" value="info"/>
- </module>
- <module name="OperatorWrap">
- <property name="severity" value="info"/>
- </module>
- <module name="ParenPad">
- <property name="severity" value="info"/>
- </module>
- <module name="TypecastParenPad">
- <property name="tokens" value="RPAREN,TYPECAST"/>
- <property name="severity" value="info"/>
- </module>
- <module name="WhitespaceAfter">
- <property name="severity" value="info"/>
- </module>
- <module name="WhitespaceAround">
- <property name="tokens"
- value="ASSIGN,BAND,BAND_ASSIGN,BOR,BOR_ASSIGN,BSR,BSR_ASSIGN,
- BXOR,BXOR_ASSIGN,COLON,DIV,DIV_ASSIGN,EQUAL,GE,GT,LAND,LE,LITERAL_ASSERT,
-
LITERAL_CATCH,LITERAL_DO,LITERAL_ELSE,LITERAL_FINALLY,LITERAL_FOR,LITERAL_IF,
-
LITERAL_RETURN,LITERAL_SYNCHRONIZED,LITERAL_TRY,LITERAL_WHILE,LOR,LT,MINUS,MINUS_ASSIGN,
- MOD,MOD_ASSIGN,NOT_EQUAL,PLUS,PLUS_ASSIGN,QUESTION,SL,SLIST,SL_ASSIGN,SR,
- SR_ASSIGN,STAR,STAR_ASSIGN,LITERAL_ASSERT,TYPE_EXTENSION_AND"/>
- <property name="severity" value="info"/>
- </module>
- </module>
-
- <module name="TreeWalker">
- <!-- Modifier Checks -->
- <!-- See
http://checkstyle.sourceforge.net/config_modifier.html -->
- <module name="ModifierOrder">
- <property name="severity" value="error"/>
- </module>
- <module name="RedundantModifier">
- <property name="severity" value="info"/>
- </module>
-
- <!-- Checks for blocks. You know, those {}'s -->
- <!--
http://checkstyle.sourceforge.net/config_blocks.html -->
- <module name="AvoidNestedBlocks">
- <property name="allowInSwitchCase" value="true"/>
- <property name="severity" value="error"/>
- </module>
- <module name="EmptyBlock">
- <property name="option" value="text"/>
- <property name="severity" value="error"/>
- </module>
- <module name="LeftCurly">
- <property name="severity" value="error"/>
- </module>
- <module name="NeedBraces">
- <property name="severity" value="error"/>
- </module>
- <module name="RightCurly">
- <property name="severity" value="error"/>
- </module>
- </module>
-
- <module name="TreeWalker">
- <!-- Checks for common coding problems -->
- <!-- See
http://checkstyle.sourceforge.net/config_coding.html -->
- <module name="CovariantEquals">
- <property name="severity" value="error"/>
- </module>
- <module name="DoubleCheckedLocking">
- <property name="severity" value="error"/>
- </module>
- <module name="EmptyStatement">
- <property name="severity" value="error"/>
- </module>
- <module name="EqualsHashCode">
- <property name="severity" value="error"/>
- </module>
- <module name="IllegalInstantiation">
- <property name="severity" value="error"/>
- </module>
- <module name="InnerAssignment">
- <property name="severity" value="error"/>
- </module>
- <module name="MagicNumber">
- <property name="ignoreNumbers" value="-1, 0, 1"/>
- <property name="severity" value="warning"/>
- <!-- TODO <property name="severity"
value="error"/> -->
- </module>
- <module name="MissingSwitchDefault">
- <property name="severity" value="error"/>
- </module>
- <module name="ModifiedControlVariable">
- <property name="severity" value="error"/>
- </module>
- <module name="RedundantThrows">
- <property name="severity" value="warning"/>
- <!-- TODO <property name="severity"
value="error"/>-->
- </module>
- <module name="SimplifyBooleanExpression">
- <property name="severity" value="error"/>
- </module>
- <module name="SimplifyBooleanReturn">
- <property name="severity" value="error"/>
- </module>
- <module name="StringLiteralEquality">
- <property name="severity" value="error"/>
- </module>
- <module name="NestedIfDepth">
- <property name="max" value="3"/>
- <property name="severity" value="warning"/>
- </module>
- <module name="NestedTryDepth">
- <property name="max" value="3"/>
- <property name="severity" value="error"/>
- </module>
- <module name="IllegalCatch">
- <property name="illegalClassNames"
- value="java.lang.Throwable, java.lang.RuntimeException,
java.lang.Error"/>
- <property name="severity" value="error"/>
- </module>
- <module name="IllegalThrows"/>
- <module name="ParameterAssignment">
- <property name="severity" value="warning"/>
- <!-- TODO <property name="severity"
value="error"/>-->
- </module>
- <module name="DefaultComesLast">
- <property name="severity" value="error"/>
- </module>
- <module name="FallThrough">
- <property name="severity" value="error"/>
- </module>
- <module name="MultipleVariableDeclarations">
- <property name="severity" value="error"/>
- </module>
- <module name="UnnecessaryParentheses">
- <property name="severity" value="info"/>
- </module>
- </module>
-
- <module name="TreeWalker">
- <!-- Checks for class design -->
- <!-- See
http://checkstyle.sourceforge.net/config_design.html -->
- <module name="FinalClass">
- <property name="severity" value="error"/>
- </module>
- <module name="HideUtilityClassConstructor">
- <property name="severity" value="error"/>
- </module>
- <module name="InterfaceIsType">
- <property name="severity" value="warning"/>
- </module>
- <module name="VisibilityModifier">
- <property name="packageAllowed" value="true"/>
- <property name="protectedAllowed" value="true"/>
- <property name="severity" value="error"/>
- </module>
- </module>
-
- <module name="TreeWalker">
- <!-- Miscellaneous other checks. -->
- <!-- See
http://checkstyle.sourceforge.net/config_metrics.html -->
- <module name="CyclomaticComplexity">
- <property name="severity" value="warning"/>
- </module>
-
- <!-- Miscellaneous other checks. -->
- <!-- See
http://checkstyle.sf.net/config_misc.html -->
- <module name="ArrayTypeStyle">
- <property name="severity" value="error"/>
- </module>
- <module name="UpperEll">
- <property name="severity" value="error"/>
- </module>
- <module name="ParameterName">
- <property name="severity" value="error"/>
- </module>
- <module name="DeclarationOrder">
- <property name="severity" value="error"/>
- </module>
- <module name="FallThrough">
- <property name="severity" value="error"/>
- </module>
- <module name="HiddenField">
- <property name="tokens" value="VARIABLE_DEF"/>
- <property name="severity" value="info"/>
- </module>
- <module name="UncommentedMain">
- <property name="severity" value="warning"/>
- </module>
- <module name="TodoComment">
- <property name="format" value="TODO"/>
- <property name="severity" value="warning"/>
- </module>
-
- <module name="Indentation">
- <property name="severity" value="error"/>
- </module>
- </module>
-</module>
Copied:
root/checkstyle/src/main/resources/exadel-richfaces-checkstyle/richfaces-checkstyle.xml
(from rev 16052,
root/checkstyle/src/main/resources/exadel-richfaces-checkstyle/exadel+sun-checkstyle.xml)
===================================================================
---
root/checkstyle/src/main/resources/exadel-richfaces-checkstyle/richfaces-checkstyle.xml
(rev 0)
+++
root/checkstyle/src/main/resources/exadel-richfaces-checkstyle/richfaces-checkstyle.xml 2009-12-03
15:35:12 UTC (rev 16054)
@@ -0,0 +1,316 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<!DOCTYPE module PUBLIC "-//Puppy Crawl//DTD Check Configuration 1.2//EN"
+ "http://www.puppycrawl.com/dtds/configuration_1_2.dtd">
+<module name="Checker">
+ <property name="severity" value="error"/>
+ <module name="Translation"/>
+ <!--<module name="FileLength">-->
+ <!--<property name="max" value="1000"/>-->
+ <!--<property name="severity"
value="warning"/>-->
+ <!--</module>-->
+
+ <module name="TreeWalker">
+ <property name="severity" value="error"/>
+
+ <!-- Checks for Javadoc comments. -->
+ <!-- See
http://checkstyle.sourceforge.net/config_javadoc.html -->
+ <module name="JavadocMethod">
+ <property name="scope" value="public"/>
+ <property name="severity" value="warning"/>
+ </module>
+ <module name="JavadocType">
+ <property name="severity" value="warning"/>
+ </module>
+ <module name="JavadocVariable">
+ <property name="scope" value="public"/>
+ <property name="severity" value="warning"/>
+ </module>
+ <module name="JavadocStyle">
+ <property name="checkEmptyJavadoc" value="true"/>
+ <property name="checkFirstSentence"
value="false"/>
+ <property name="severity" value="warning"/>
+ </module>
+ </module>
+
+ <module name="TreeWalker">
+ <!-- Checks for Naming Conventions. -->
+ <!-- See
http://checkstyle.sourceforge.net/config_naming.html -->
+ <module name="ConstantName">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="LocalFinalVariableName">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="LocalVariableName">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="MemberName">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="MethodName">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="PackageName">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="ParameterName">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="StaticVariableName">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="TypeName">
+ <property name="severity" value="error"/>
+ </module>
+ </module>
+
+ <module name="TreeWalker">
+ <!-- Checks for imports -->
+ <!-- See
http://checkstyle.sourceforge.net/config_import.html -->
+ <module name="AvoidStarImport">
+ <property name="severity" value="warning"/>
+ </module>
+ <module name="IllegalImport">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="RedundantImport">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="UnusedImports">
+ <property name="severity" value="error"/>
+ </module>
+ </module>
+
+ <module name="TreeWalker">
+ <!-- Checks for Size Violations. -->
+ <!-- See
http://checkstyle.sourceforge.net/config_sizes.html -->
+ <module name="LineLength">
+ <property name="max" value="120"/>
+ <property name="severity" value="warning"/>
+ </module>
+ <module name="MethodLength">
+ <property name="tokens" value="METHOD_DEF"/>
+ <property name="max" value="60"/>
+ <!-- I think must be 30 -->
+ <property name="countEmpty" value="false"/>
+ <property name="severity" value="warning"/>
+ </module>
+ <module name="AnonInnerLength">
+ <property name="max" value="40"/>
+ <property name="severity" value="error"/>
+ </module>
+ <module name="ParameterNumber">
+ <property name="severity" value="warning"/>
+ </module>
+ </module>
+
+ <module name="TreeWalker">
+ <!-- Checks for whitespace -->
+ <!-- See
http://checkstyle.sourceforge.net/config_whitespace.html -->
+ <!--<module name="TabCharacter"/>-->
+ <module name="EmptyForIteratorPad">
+ <property name="severity" value="info"/>
+ </module>
+ <module name="MethodParamPad">
+ <property name="severity" value="info"/>
+ </module>
+ <module name="NoWhitespaceAfter">
+ <property name="severity" value="info"/>
+ </module>
+ <module name="NoWhitespaceBefore">
+ <property name="severity" value="info"/>
+ </module>
+ <module name="OperatorWrap">
+ <property name="severity" value="info"/>
+ </module>
+ <module name="ParenPad">
+ <property name="severity" value="info"/>
+ </module>
+ <module name="TypecastParenPad">
+ <property name="tokens" value="RPAREN,TYPECAST"/>
+ <property name="severity" value="info"/>
+ </module>
+ <module name="WhitespaceAfter">
+ <property name="severity" value="info"/>
+ </module>
+ <module name="WhitespaceAround">
+ <property name="tokens"
+ value="ASSIGN,BAND,BAND_ASSIGN,BOR,BOR_ASSIGN,BSR,BSR_ASSIGN,
+ BXOR,BXOR_ASSIGN,COLON,DIV,DIV_ASSIGN,EQUAL,GE,GT,LAND,LE,LITERAL_ASSERT,
+
LITERAL_CATCH,LITERAL_DO,LITERAL_ELSE,LITERAL_FINALLY,LITERAL_FOR,LITERAL_IF,
+
LITERAL_RETURN,LITERAL_SYNCHRONIZED,LITERAL_TRY,LITERAL_WHILE,LOR,LT,MINUS,MINUS_ASSIGN,
+ MOD,MOD_ASSIGN,NOT_EQUAL,PLUS,PLUS_ASSIGN,QUESTION,SL,SLIST,SL_ASSIGN,SR,
+ SR_ASSIGN,STAR,STAR_ASSIGN,LITERAL_ASSERT,TYPE_EXTENSION_AND"/>
+ <property name="severity" value="info"/>
+ </module>
+ </module>
+
+ <module name="TreeWalker">
+ <!-- Modifier Checks -->
+ <!-- See
http://checkstyle.sourceforge.net/config_modifier.html -->
+ <module name="ModifierOrder">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="RedundantModifier">
+ <property name="severity" value="info"/>
+ </module>
+
+ <!-- Checks for blocks. You know, those {}'s -->
+ <!--
http://checkstyle.sourceforge.net/config_blocks.html -->
+ <module name="AvoidNestedBlocks">
+ <property name="allowInSwitchCase" value="true"/>
+ <property name="severity" value="error"/>
+ </module>
+ <module name="EmptyBlock">
+ <property name="option" value="text"/>
+ <property name="severity" value="error"/>
+ </module>
+ <module name="LeftCurly">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="NeedBraces">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="RightCurly">
+ <property name="severity" value="error"/>
+ </module>
+ </module>
+
+ <module name="TreeWalker">
+ <!-- Checks for common coding problems -->
+ <!-- See
http://checkstyle.sourceforge.net/config_coding.html -->
+ <module name="CovariantEquals">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="DoubleCheckedLocking">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="EmptyStatement">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="EqualsHashCode">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="IllegalInstantiation">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="InnerAssignment">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="MagicNumber">
+ <property name="ignoreNumbers" value="-1, 0, 1"/>
+ <property name="severity" value="warning"/>
+ <!-- TODO <property name="severity"
value="error"/> -->
+ </module>
+ <module name="MissingSwitchDefault">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="ModifiedControlVariable">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="RedundantThrows">
+ <property name="severity" value="warning"/>
+ <!-- TODO <property name="severity"
value="error"/>-->
+ </module>
+ <module name="SimplifyBooleanExpression">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="SimplifyBooleanReturn">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="StringLiteralEquality">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="NestedIfDepth">
+ <property name="max" value="3"/>
+ <property name="severity" value="warning"/>
+ </module>
+ <module name="NestedTryDepth">
+ <property name="max" value="3"/>
+ <property name="severity" value="error"/>
+ </module>
+ <module name="IllegalCatch">
+ <property name="illegalClassNames"
+ value="java.lang.Throwable, java.lang.RuntimeException,
java.lang.Error"/>
+ <property name="severity" value="error"/>
+ </module>
+ <module name="IllegalThrows"/>
+ <module name="ParameterAssignment">
+ <property name="severity" value="warning"/>
+ <!-- TODO <property name="severity"
value="error"/>-->
+ </module>
+ <module name="DefaultComesLast">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="FallThrough">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="MultipleVariableDeclarations">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="UnnecessaryParentheses">
+ <property name="severity" value="info"/>
+ </module>
+ </module>
+
+ <module name="TreeWalker">
+ <!-- Checks for class design -->
+ <!-- See
http://checkstyle.sourceforge.net/config_design.html -->
+ <module name="FinalClass">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="HideUtilityClassConstructor">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="InterfaceIsType">
+ <property name="severity" value="warning"/>
+ </module>
+ <module name="VisibilityModifier">
+ <property name="packageAllowed" value="true"/>
+ <property name="protectedAllowed" value="true"/>
+ <property name="severity" value="error"/>
+ </module>
+ </module>
+
+ <module name="TreeWalker">
+ <!-- Miscellaneous other checks. -->
+ <!-- See
http://checkstyle.sourceforge.net/config_metrics.html -->
+ <module name="CyclomaticComplexity">
+ <property name="severity" value="warning"/>
+ </module>
+
+ <!-- Miscellaneous other checks. -->
+ <!-- See
http://checkstyle.sf.net/config_misc.html -->
+ <module name="ArrayTypeStyle">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="UpperEll">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="ParameterName">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="DeclarationOrder">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="FallThrough">
+ <property name="severity" value="error"/>
+ </module>
+ <module name="HiddenField">
+ <property name="tokens" value="VARIABLE_DEF"/>
+ <property name="severity" value="info"/>
+ </module>
+ <module name="UncommentedMain">
+ <property name="severity" value="warning"/>
+ </module>
+ <module name="TodoComment">
+ <property name="format" value="TODO"/>
+ <property name="severity" value="warning"/>
+ </module>
+
+ <module name="Indentation">
+ <property name="severity" value="error"/>
+ </module>
+ </module>
+</module>