Automatic code cleanup.

PiperOrigin-RevId: 644166840
google · Jun 18, 2024 · ddeea7e · ddeea7e
1 parent a3fdf64
commit ddeea7e
Show file tree

Hide file tree

Showing 7 changed files with 16 additions and 16 deletions.
diff --git a/check_api/src/main/java/com/google/errorprone/matchers/Matchers.java b/check_api/src/main/java/com/google/errorprone/matchers/Matchers.java
@@ -594,7 +594,7 @@ public static Matcher<ExpressionTree> nonNullLiteral() {
         case DOUBLE_LITERAL:
         case BOOLEAN_LITERAL:
         case CHAR_LITERAL:
-          // fall through
+        // fall through
         case STRING_LITERAL:
           return true;
         default:

diff --git a/check_api/src/main/java/com/google/errorprone/util/ASTHelpers.java b/check_api/src/main/java/com/google/errorprone/util/ASTHelpers.java
@@ -2074,25 +2074,25 @@ public Type visitBinary(BinaryTree tree, Void unused) {
       Type leftType = checkNotNull(getType(tree.getLeftOperand()));
       Type rightType = checkNotNull(getType(tree.getRightOperand()));
       switch (tree.getKind()) {
-          // The addition and subtraction operators for numeric types + and - (§15.18.2)
+        // The addition and subtraction operators for numeric types + and - (§15.18.2)
         case PLUS:
           // If either operand is of string type, string concatenation is performed.
           Type stringType = state.getSymtab().stringType;
           if (isSameType(stringType, leftType, state) || isSameType(stringType, rightType, state)) {
             return stringType;
           }
-          // Fall through.
+        // Fall through.
         case MINUS:
-          // The multiplicative operators *, /, and % (§15.17)
+        // The multiplicative operators *, /, and % (§15.17)
         case MULTIPLY:
         case DIVIDE:
         case REMAINDER:
-          // The numerical comparison operators <, <=, >, and >= (§15.20.1)
+        // The numerical comparison operators <, <=, >, and >= (§15.20.1)
         case LESS_THAN:
         case LESS_THAN_EQUAL:
         case GREATER_THAN:
         case GREATER_THAN_EQUAL:
-          // The integer bitwise operators &, ^, and |
+        // The integer bitwise operators &, ^, and |
         case AND:
         case XOR:
         case OR:

diff --git a/check_api/src/test/java/com/google/errorprone/util/testdata/TargetTypeTest.java b/check_api/src/test/java/com/google/errorprone/util/testdata/TargetTypeTest.java
@@ -438,21 +438,21 @@ void testAssert() {
   void testSwitch(int anInt, String aString) {
     final int detectInt = 0;
     switch (anInt) {
-        // BUG: Diagnostic contains: int
+      // BUG: Diagnostic contains: int
       case detectInt:
         break;
     }
 
     final byte detectByte = 0;
     switch (anInt) {
-        // BUG: Diagnostic contains: int
+      // BUG: Diagnostic contains: int
       case detectByte:
         break;
     }
 
     final String detectString = "";
     switch (aString) {
-        // BUG: Diagnostic contains: java.lang.String
+      // BUG: Diagnostic contains: java.lang.String
       case detectString:
         break;
     }

diff --git a/core/src/main/java/com/google/errorprone/bugpatterns/CanBeStaticAnalyzer.java b/core/src/main/java/com/google/errorprone/bugpatterns/CanBeStaticAnalyzer.java
@@ -80,10 +80,10 @@ public void visitIdent(JCTree.JCIdent tree) {
     }
     switch (sym.getKind()) {
       case TYPE_PARAMETER:
-        // declaring a class as non-static just to access a type parameter is silly -
-        // why not just re-declare the type parameter instead of capturing it?
-        // TODO(cushon): consider making the suggestion anyways, maybe with a fix?
-        // fall through
+      // declaring a class as non-static just to access a type parameter is silly -
+      // why not just re-declare the type parameter instead of capturing it?
+      // TODO(cushon): consider making the suggestion anyways, maybe with a fix?
+      // fall through
       case FIELD:
         if (!isOwnedBy(sym, owner, state.getTypes())) {
           canPossiblyBeStatic = false;

diff --git a/core/src/main/java/com/google/errorprone/bugpatterns/StaticQualifiedUsingExpression.java b/core/src/main/java/com/google/errorprone/bugpatterns/StaticQualifiedUsingExpression.java
@@ -61,7 +61,7 @@ public Description matchMemberSelect(MemberSelectTree tree, VisitorState state)
             || sym.getSimpleName().contentEquals("super")) {
           return NO_MATCH;
         }
-        // fall through
+      // fall through
       case ENUM_CONSTANT:
       case METHOD:
         if (!isStatic(sym)) {

diff --git a/core/src/main/java/com/google/errorprone/bugpatterns/UnnecessaryBoxedVariable.java b/core/src/main/java/com/google/errorprone/bugpatterns/UnnecessaryBoxedVariable.java
@@ -108,7 +108,7 @@ private Optional<Description> handleVariable(
             || state.getPath().getParentPath().getLeaf() instanceof LambdaExpressionTree) {
           return Optional.empty();
         }
-        // Fall through.
+      // Fall through.
       case LOCAL_VARIABLE:
         if (!variableMatches(tree, state)) {
           return Optional.empty();

diff --git a/core/src/main/java/com/google/errorprone/bugpatterns/nullness/NullnessUtils.java b/core/src/main/java/com/google/errorprone/bugpatterns/nullness/NullnessUtils.java
@@ -385,7 +385,7 @@ private static boolean isTypeUse(String className) {
       case "org.checkerframework.checker.nullness.qual.Nullable":
       case "org.jspecify.annotations.NonNull":
       case "org.jspecify.annotations.Nullable":
-        // We break these strings to avoid having them rewritten by Copybara.
+      // We break these strings to avoid having them rewritten by Copybara.
       case "org.jspecify.null" + "ness.NonNull":
       case "org.jspecify.null" + "ness.Nullable":
         return true;