Issue #12615: removes unnecessary code for EmptyBlock

checkstyle · Jan 11, 2023 · 8bfa87c · 8bfa87c
1 parent affd2bd
commit 8bfa87c
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 11 deletions.
diff --git a/config/pitest-suppressions/pitest-blocks-suppressions.xml b/config/pitest-suppressions/pitest-blocks-suppressions.xml
@@ -1,14 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <suppressedMutations>
-  <mutation unstable="false">
-    <sourceFile>EmptyBlockCheck.java</sourceFile>
-    <mutatedClass>com.puppycrawl.tools.checkstyle.checks.blocks.EmptyBlockCheck</mutatedClass>
-    <mutatedMethod>visitToken</mutatedMethod>
-    <mutator>org.pitest.mutationtest.engine.gregor.mutators.NonVoidMethodCallMutator</mutator>
-    <description>removed call to com/puppycrawl/tools/checkstyle/api/DetailAST::getText</description>
-    <lineContent>ast.getText());</lineContent>
-  </mutation>
-
   <mutation unstable="false">
     <sourceFile>RightCurlyCheck.java</sourceFile>
     <mutatedClass>com.puppycrawl.tools.checkstyle.checks.blocks.RightCurlyCheck</mutatedClass>

diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/blocks/EmptyBlockCheck.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/blocks/EmptyBlockCheck.java
@@ -260,8 +260,7 @@ public void visitToken(DetailAST ast) {
                 }
                 if (emptyBlock) {
                     log(leftCurly,
-                        MSG_KEY_BLOCK_NO_STATEMENT,
-                        ast.getText());
+                        MSG_KEY_BLOCK_NO_STATEMENT);
                 }
             }
             else if (!hasText(leftCurly)) {