diff options
Diffstat (limited to 'checkstyle.xml')
-rw-r--r-- | checkstyle.xml | 29 |
1 files changed, 25 insertions, 4 deletions
diff --git a/checkstyle.xml b/checkstyle.xml index 00dfe5b3f..6c232edce 100644 --- a/checkstyle.xml +++ b/checkstyle.xml @@ -22,7 +22,7 @@ <module name = "Checker"> <property name="charset" value="UTF-8"/> - <property name="severity" value="warning"/> + <property name="severity" value="error"/> <property name="fileExtensions" value="java, properties, xml"/> <!-- Checks for whitespace --> @@ -48,6 +48,7 @@ <module name="LineLength"> <property name="max" value="120"/> <property name="ignorePattern" value="^package.*|^import.*|a href|href|http://|https://|ftp://"/> + <property name="severity" value="error"/> </module> <module name="AvoidStarImport"/> <module name="OneTopLevelClass"/> @@ -57,10 +58,13 @@ <property name="tokens" value="LITERAL_TRY, LITERAL_FINALLY, LITERAL_IF, LITERAL_ELSE, LITERAL_SWITCH"/> </module> - <module name="NeedBraces"/> + <module name="NeedBraces"> + <property name="severity" value="error"/> + </module> <module name="LeftCurly"/> <module name="RightCurly"> <property name="id" value="RightCurlySame"/> + <property name="severity" value="error"/> <property name="tokens" value="LITERAL_TRY, LITERAL_CATCH, LITERAL_FINALLY, LITERAL_IF, LITERAL_ELSE, LITERAL_DO"/> @@ -68,6 +72,7 @@ <module name="RightCurly"> <property name="id" value="RightCurlyAlone"/> <property name="option" value="alone"/> + <property name="severity" value="error"/> <property name="tokens" value="CLASS_DEF, METHOD_DEF, CTOR_DEF, LITERAL_FOR, LITERAL_WHILE, STATIC_INIT, INSTANCE_INIT"/> @@ -81,6 +86,7 @@ value="WhitespaceAround: ''{0}'' is not followed by whitespace. Empty blocks may only be represented as '{}' when not part of a multi-block statement (4.1.3)"/> <message key="ws.notPreceded" value="WhitespaceAround: ''{0}'' is not preceded with whitespace."/> + <property name="severity" value="error"/> </module> <module name="OneStatementPerLine"/> <module name="MultipleVariableDeclarations"/> @@ -90,7 +96,9 @@ <module name="MissingSwitchDefault"/> <module name="FallThrough"/> <module name="UpperEll"/> - <module name="ModifierOrder"/> + <module name="ModifierOrder"> + <property name="severity" value="error"/> + </module> <module name="EmptyLineSeparator"> <property name="allowNoEmptyLineBetweenFields" value="true"/> </module> @@ -129,16 +137,25 @@ <module name="TypeName"> <message key="name.invalidPattern" value="Type name ''{0}'' must match pattern ''{1}''."/> + <property name="severity" value="error"/> </module> <module name="MemberName"> <property name="format" value="^m[A-Z][a-zA-Z0-9]*$"/> <message key="name.invalidPattern" value="Member name ''{0}'' must match pattern ''{1}''."/> + <property name="severity" value="error"/> + </module> + <module name="StaticVariableName"> + <property name="severity" value="error"/> + <property name="format" value="^s[A-Z0-9][a-zA-Z0-9]*$"/> + <message key="name.invalidPattern" + value="Static field names start with s and are camel-cased. (e.g. sFooBar)"/> </module> <module name="ParameterName"> <property name="format" value="^[a-z]([a-z0-9][a-zA-Z0-9]*)?$"/> <message key="name.invalidPattern" value="Parameter name ''{0}'' must match pattern ''{1}''."/> + <property name="severity" value="error"/> </module> <module name="CatchParameterName"> <property name="format" value="^[a-z]([a-z0-9][a-zA-Z0-9]*)?$"/> @@ -150,6 +167,7 @@ <property name="format" value="^[a-z]([a-zA-Z0-9]*)?$"/> <message key="name.invalidPattern" value="Local variable name ''{0}'' must match pattern ''{1}''."/> + <property name="severity" value="error"/> </module> <module name="ClassTypeParameterName"> <property name="format" value="(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)"/> @@ -160,6 +178,7 @@ <property name="format" value="(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)"/> <message key="name.invalidPattern" value="Method type name ''{0}'' must match pattern ''{1}''."/> + <property name="severity" value="error"/> </module> <module name="InterfaceTypeParameterName"> <property name="format" value="(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)"/> @@ -176,6 +195,7 @@ value="GenericWhitespace ''{0}'' should followed by whitespace."/> <message key="ws.notPreceded" value="GenericWhitespace ''{0}'' is not preceded with whitespace."/> + <property name="severity" value="error"/> </module> <module name="Indentation"> <property name="basicOffset" value="4"/> @@ -187,7 +207,7 @@ </module> <module name="AbbreviationAsWordInName"> <property name="ignoreFinal" value="false"/> - <property name="allowedAbbreviationLength" value="2"/> + <property name="allowedAbbreviationLength" value="3"/> </module> <module name="OverloadMethodsDeclarationOrder"/> <module name="VariableDeclarationUsageDistance"/> @@ -202,6 +222,7 @@ <property name="tokens" value="COMMA, SEMI, POST_INC, POST_DEC, DOT, ELLIPSIS, METHOD_REF"/> <property name="allowLineBreaks" value="true"/> + <property name="severity" value="error"/> </module> <module name="ParenPad"/> <module name="OperatorWrap"> |