From 95c1a081e0868fc3ac9faf05d09ca8b2bb06a5cc Mon Sep 17 00:00:00 2001 From: alexkravin Date: Thu, 22 Jan 2015 16:26:45 +0400 Subject: [PATCH] Changed indentation of switch-case, checkstyle, issue #512 --- .../tools/checkstyle/TreeWalker.java | 24 ++++----- .../tools/checkstyle/XMLLogger.java | 52 +++++++++---------- 2 files changed, 38 insertions(+), 38 deletions(-) diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/TreeWalker.java b/src/main/java/com/puppycrawl/tools/checkstyle/TreeWalker.java index 935f190af..19aa0b96a 100755 --- a/src/main/java/com/puppycrawl/tools/checkstyle/TreeWalker.java +++ b/src/main/java/com/puppycrawl/tools/checkstyle/TreeWalker.java @@ -652,12 +652,12 @@ public final class TreeWalker private static DetailAST createCommentAstFromToken(Token token) { switch (token.getType()) { - case TokenTypes.SINGLE_LINE_COMMENT: - return createSlCommentNode(token); - case TokenTypes.BLOCK_COMMENT_BEGIN: - return createBlockCommentNode(token); - default: - throw new IllegalArgumentException("Unknown comment type"); + case TokenTypes.SINGLE_LINE_COMMENT: + return createSlCommentNode(token); + case TokenTypes.BLOCK_COMMENT_BEGIN: + return createBlockCommentNode(token); + default: + throw new IllegalArgumentException("Unknown comment type"); } } @@ -747,12 +747,12 @@ public final class TreeWalker int columns = initialColumnsCnt; for (char c : text.toCharArray()) { switch (c) { - case '\n': - lines++; - columns = 0; - break; - default: - columns++; + case '\n': + lines++; + columns = 0; + break; + default: + columns++; } } return new SimpleEntry(lines, columns); diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/XMLLogger.java b/src/main/java/com/puppycrawl/tools/checkstyle/XMLLogger.java index 2e7999391..06366b016 100644 --- a/src/main/java/com/puppycrawl/tools/checkstyle/XMLLogger.java +++ b/src/main/java/com/puppycrawl/tools/checkstyle/XMLLogger.java @@ -174,32 +174,32 @@ public class XMLLogger for (int i = 0; i < value.length(); i++) { final char c = value.charAt(i); switch (c) { - case '<': - sb.append("<"); - break; - case '>': - sb.append(">"); - break; - case '\'': - sb.append("'"); - break; - case '\"': - sb.append("""); - break; - case '&': - final int nextSemi = value.indexOf(";", i); - if ((nextSemi < 0) - || !isReference(value.substring(i, nextSemi + 1))) - { - sb.append("&"); - } - else { - sb.append('&'); - } - break; - default: - sb.append(c); - break; + case '<': + sb.append("<"); + break; + case '>': + sb.append(">"); + break; + case '\'': + sb.append("'"); + break; + case '\"': + sb.append("""); + break; + case '&': + final int nextSemi = value.indexOf(";", i); + if ((nextSemi < 0) + || !isReference(value.substring(i, nextSemi + 1))) + { + sb.append("&"); + } + else { + sb.append('&'); + } + break; + default: + sb.append(c); + break; } } return sb.toString();