diff --git a/build.xml b/build.xml
index 7227abea1..1728aa288 100644
--- a/build.xml
+++ b/build.xml
@@ -2,6 +2,9 @@
+
+
+
diff --git a/docs/checkstyle_checks.xml b/docs/checkstyle_checks.xml
index 6e9599977..a01572ca3 100644
--- a/docs/checkstyle_checks.xml
+++ b/docs/checkstyle_checks.xml
@@ -107,5 +107,13 @@
+
+
+
+
+
+
+
+
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/Checker.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/Checker.java
index 4c2d618d5..cff915a60 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/Checker.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/Checker.java
@@ -146,7 +146,7 @@ public class Checker extends AutomaticBean
/** @see AutomaticBean */
public void finishLocalSetup()
- throws CheckstyleException
+ throws CheckstyleException
{
final Locale locale = new Locale(mLocaleLanguage, mLocaleCountry);
LocalizedMessage.setLocale(locale);
@@ -169,7 +169,7 @@ public class Checker extends AutomaticBean
* @see com.puppycrawl.tools.checkstyle.api.AutomaticBean
*/
protected void setupChild(Configuration aChildConf)
- throws CheckstyleException
+ throws CheckstyleException
{
final String name = aChildConf.getName();
try {
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/ConfigurationLoader.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/ConfigurationLoader.java
index 0b9b4b57e..0cfa638d4 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/ConfigurationLoader.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/ConfigurationLoader.java
@@ -89,7 +89,7 @@ public final class ConfigurationLoader
String aLocalName,
String aQName,
Attributes aAtts)
- throws SAXException
+ throws SAXException
{
// TODO: debug logging for support puposes
if (aQName.equals(MODULE)) {
@@ -266,9 +266,8 @@ public final class ConfigurationLoader
* Code copied from ant -
* http://cvs.apache.org/viewcvs/jakarta-ant/src/main/org/apache/tools/ant/ProjectHelper.java
*/
- static String replaceProperties(
- String aValue, PropertyResolver aProps)
- throws CheckstyleException
+ static String replaceProperties(String aValue, PropertyResolver aProps)
+ throws CheckstyleException
{
if (aValue == null) {
return null;
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/PackageNamesLoader.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/PackageNamesLoader.java
index 62938e5fe..6c2fb44d2 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/PackageNamesLoader.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/PackageNamesLoader.java
@@ -78,7 +78,7 @@ public final class PackageNamesLoader
String aLocalName,
String aQName,
Attributes aAtts)
- throws SAXException
+ throws SAXException
{
if (aQName.equals("package")) {
//push package name
@@ -88,7 +88,7 @@ public final class PackageNamesLoader
}
mPackageStack.push(name);
}
- }
+ }
/**
* Creates a full package name from the package names on the stack.
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/TreeWalker.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/TreeWalker.java
index 0810afe6b..736f981f1 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/TreeWalker.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/TreeWalker.java
@@ -199,7 +199,7 @@ public final class TreeWalker
* @see com.puppycrawl.tools.checkstyle.api.AutomaticBean
*/
public void setupChild(Configuration aChildConf)
- throws CheckstyleException
+ throws CheckstyleException
{
// TODO: improve the error handing
final String name = aChildConf.getName();
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/api/AutomaticBean.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/api/AutomaticBean.java
index c0a94a9b0..240e569fb 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/api/AutomaticBean.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/api/AutomaticBean.java
@@ -209,7 +209,7 @@ public class AutomaticBean
* @see Contextualizable
*/
public final void contextualize(Context aContext)
- throws CheckstyleException
+ throws CheckstyleException
{
// TODO: debug log messages
final String[] attributes = aContext.getAttributeNames();
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/EmptyBlockCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/EmptyBlockCheck.java
index 61d5eba31..4b64f437c 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/EmptyBlockCheck.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/EmptyBlockCheck.java
@@ -127,7 +127,7 @@ public class EmptyBlockCheck
final String txt = lines[slistLineNo - 1]
.substring(slistColNo + 1, rcurlyColNo);
if (txt.trim().length() != 0) {
- retVal = true;
+ retVal = true;
}
}
else {
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/EmptyStatementCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/EmptyStatementCheck.java
index 240b828b4..94bd364ac 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/EmptyStatementCheck.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/EmptyStatementCheck.java
@@ -38,15 +38,15 @@ import com.puppycrawl.tools.checkstyle.api.DetailAST;
*/
public class EmptyStatementCheck extends Check
{
- /** @see com.puppycrawl.tools.checkstyle.api.Check */
- public int[] getDefaultTokens()
- {
- return new int[] {TokenTypes.EMPTY_STAT};
- }
+ /** @see com.puppycrawl.tools.checkstyle.api.Check */
+ public int[] getDefaultTokens()
+ {
+ return new int[] {TokenTypes.EMPTY_STAT};
+ }
- /** @see com.puppycrawl.tools.checkstyle.api.Check */
- public void visitToken(DetailAST aAST)
- {
- log(aAST.getLineNo(), aAST.getColumnNo(), "empty.statement");
- }
+ /** @see com.puppycrawl.tools.checkstyle.api.Check */
+ public void visitToken(DetailAST aAST)
+ {
+ log(aAST.getLineNo(), aAST.getColumnNo(), "empty.statement");
+ }
}
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/EqualsHashCodeCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/EqualsHashCodeCheck.java
index cefd80979..ec55e1977 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/EqualsHashCodeCheck.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/EqualsHashCodeCheck.java
@@ -89,7 +89,7 @@ public class EqualsHashCodeCheck
&& modifiers.branchContains(TokenTypes.LITERAL_PUBLIC)
&& parameters.getChildCount() == 1
&& isObjectParam(parameters.getFirstChild())
- )
+ )
{
mObjBlockEquals.put(aAST.getParent(), aAST);
}
@@ -112,17 +112,17 @@ public class EqualsHashCodeCheck
final AST modifiers = aFirstChild.getFirstChild();
AST type = modifiers.getNextSibling();
switch (type.getFirstChild().getType()) {
- case TokenTypes.LITERAL_BOOLEAN:
- case TokenTypes.LITERAL_BYTE:
- case TokenTypes.LITERAL_CHAR:
- case TokenTypes.LITERAL_DOUBLE:
- case TokenTypes.LITERAL_FLOAT:
- case TokenTypes.LITERAL_INT:
- case TokenTypes.LITERAL_LONG:
- case TokenTypes.LITERAL_SHORT:
- return false;
- default:
- return true;
+ case TokenTypes.LITERAL_BOOLEAN:
+ case TokenTypes.LITERAL_BYTE:
+ case TokenTypes.LITERAL_CHAR:
+ case TokenTypes.LITERAL_DOUBLE:
+ case TokenTypes.LITERAL_FLOAT:
+ case TokenTypes.LITERAL_INT:
+ case TokenTypes.LITERAL_LONG:
+ case TokenTypes.LITERAL_SHORT:
+ return false;
+ default:
+ return true;
}
}
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/HiddenFieldCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/HiddenFieldCheck.java
index a139066f8..31befca40 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/HiddenFieldCheck.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/HiddenFieldCheck.java
@@ -150,4 +150,4 @@ public class HiddenFieldCheck
}
}
}
- }
+}
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/IllegalInstantiationCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/IllegalInstantiationCheck.java
index 865e06351..f16620bb1 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/IllegalInstantiationCheck.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/IllegalInstantiationCheck.java
@@ -114,17 +114,17 @@ public class IllegalInstantiationCheck
public void visitToken(DetailAST aAST)
{
switch (aAST.getType()) {
- case TokenTypes.LITERAL_NEW:
- processLiteralNew(aAST);
- break;
- case TokenTypes.PACKAGE_DEF:
- processPackageDef(aAST);
- break;
- case TokenTypes.IMPORT:
- processImport(aAST);
- break;
- default:
- throw new IllegalArgumentException("Unknown type " + aAST);
+ case TokenTypes.LITERAL_NEW:
+ processLiteralNew(aAST);
+ break;
+ case TokenTypes.PACKAGE_DEF:
+ processPackageDef(aAST);
+ break;
+ case TokenTypes.IMPORT:
+ processImport(aAST);
+ break;
+ default:
+ throw new IllegalArgumentException("Unknown type " + aAST);
}
}
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/LeftCurlyCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/LeftCurlyCheck.java
index 80c74501d..6a4ca12f3 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/LeftCurlyCheck.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/LeftCurlyCheck.java
@@ -119,48 +119,48 @@ public class LeftCurlyCheck
final DetailAST brace;
switch (aAST.getType()) {
- case TokenTypes.CTOR_DEF :
- case TokenTypes.METHOD_DEF :
- startToken = aAST;
- brace = aAST.findFirstToken(TokenTypes.SLIST);
- break;
+ case TokenTypes.CTOR_DEF :
+ case TokenTypes.METHOD_DEF :
+ startToken = aAST;
+ brace = aAST.findFirstToken(TokenTypes.SLIST);
+ break;
- case TokenTypes.INTERFACE_DEF :
- case TokenTypes.CLASS_DEF :
- // TODO: should check for modifiers
- startToken = (DetailAST) aAST.getFirstChild().getNextSibling();
- brace = (DetailAST) aAST.getLastChild().getFirstChild();
- break;
+ case TokenTypes.INTERFACE_DEF :
+ case TokenTypes.CLASS_DEF :
+ // TODO: should check for modifiers
+ startToken = (DetailAST) aAST.getFirstChild().getNextSibling();
+ brace = (DetailAST) aAST.getLastChild().getFirstChild();
+ break;
- case TokenTypes.LITERAL_WHILE:
- case TokenTypes.LITERAL_CATCH:
- case TokenTypes.LITERAL_SYNCHRONIZED:
- case TokenTypes.LITERAL_FOR:
- case TokenTypes.LITERAL_TRY:
- case TokenTypes.LITERAL_FINALLY:
- case TokenTypes.LITERAL_DO:
- case TokenTypes.LITERAL_IF :
- startToken = aAST;
- brace = aAST.findFirstToken(TokenTypes.SLIST);
- break;
+ case TokenTypes.LITERAL_WHILE:
+ case TokenTypes.LITERAL_CATCH:
+ case TokenTypes.LITERAL_SYNCHRONIZED:
+ case TokenTypes.LITERAL_FOR:
+ case TokenTypes.LITERAL_TRY:
+ case TokenTypes.LITERAL_FINALLY:
+ case TokenTypes.LITERAL_DO:
+ case TokenTypes.LITERAL_IF :
+ startToken = aAST;
+ brace = aAST.findFirstToken(TokenTypes.SLIST);
+ break;
- case TokenTypes.LITERAL_ELSE :
- startToken = aAST;
- final DetailAST candidate = (DetailAST) aAST.getFirstChild();
- brace =
- (candidate.getType() == TokenTypes.SLIST)
- ? candidate
- : null; // silently ignore
- break;
+ case TokenTypes.LITERAL_ELSE :
+ startToken = aAST;
+ final DetailAST candidate = (DetailAST) aAST.getFirstChild();
+ brace =
+ (candidate.getType() == TokenTypes.SLIST)
+ ? candidate
+ : null; // silently ignore
+ break;
- case TokenTypes.LITERAL_SWITCH :
- startToken = aAST;
- brace = aAST.findFirstToken(TokenTypes.LCURLY);
- break;
+ case TokenTypes.LITERAL_SWITCH :
+ startToken = aAST;
+ brace = aAST.findFirstToken(TokenTypes.LCURLY);
+ break;
- default :
- startToken = null;
- brace = null;
+ default :
+ startToken = null;
+ brace = null;
}
if ((brace != null) && (startToken != null)) {
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/LineSeparatorOption.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/LineSeparatorOption.java
index c028efcee..fcae0357c 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/LineSeparatorOption.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/LineSeparatorOption.java
@@ -34,7 +34,7 @@ public final class LineSeparatorOption extends AbstractOption
/** Windows-style line separators **/
public static final LineSeparatorOption CRLF =
- new LineSeparatorOption("crlf", "\r\n");
+ new LineSeparatorOption("crlf", "\r\n");
/** Mac-style line separators **/
public static final LineSeparatorOption CR =
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/SimplifyBooleanExpressionCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/SimplifyBooleanExpressionCheck.java
index e1d342c79..42feb7c8e 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/SimplifyBooleanExpressionCheck.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/SimplifyBooleanExpressionCheck.java
@@ -68,16 +68,16 @@ public class SimplifyBooleanExpressionCheck
{
final DetailAST parent = aAST.getParent();
switch (parent.getType()) {
- case TokenTypes.NOT_EQUAL:
- case TokenTypes.EQUAL:
- case TokenTypes.LNOT:
- case TokenTypes.LOR:
- case TokenTypes.LAND:
- log(parent.getLineNo(), parent.getColumnNo(),
- "simplify.expression");
- break;
- default:
- break;
+ case TokenTypes.NOT_EQUAL:
+ case TokenTypes.EQUAL:
+ case TokenTypes.LNOT:
+ case TokenTypes.LOR:
+ case TokenTypes.LAND:
+ log(parent.getLineNo(), parent.getColumnNo(),
+ "simplify.expression");
+ break;
+ default:
+ break;
}
}
}
diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/VisibilityModifierCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/VisibilityModifierCheck.java
index aec9c8121..c04790832 100644
--- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/VisibilityModifierCheck.java
+++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/VisibilityModifierCheck.java
@@ -186,12 +186,12 @@ public class VisibilityModifierCheck
DetailAST ast = aAST.getParent();
while (ast != null) {
switch (ast.getType()) {
- case TokenTypes.INTERFACE_DEF:
- return true;
- case TokenTypes.CLASS_DEF:
- return false;
- default:
- ast = ast.getParent();
+ case TokenTypes.INTERFACE_DEF:
+ return true;
+ case TokenTypes.CLASS_DEF:
+ return false;
+ default:
+ ast = ast.getParent();
}
}
return false;