Issue checkstyle#13213: Removed //ok from matchxpath

config/checkstyle-input-suppressions.xml

@@ -248,30 +248,6 @@
             files="checks[\\/]coding[\\/]illegaltype[\\/]InputIllegalTypeWhitespaceInConfig.java"/>
   <suppress id="UnnecessaryOkComment"
             files="checks[\\/]coding[\\/]innerassignment[\\/]InputInnerAssignmentLambdaExpressions.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]coding[\\/]matchxpath[\\/]InputMatchXpath.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]coding[\\/]matchxpath[\\/]InputMatchXpath2.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]coding[\\/]matchxpath[\\/]InputMatchXpath3.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]coding[\\/]matchxpath[\\/]InputMatchXpath4.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]coding[\\/]matchxpath[\\/]InputMatchXpathAvoidInstanceCreationWithoutVar.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]coding[\\/]matchxpath[\\/]InputMatchXpathExecutableStatementCount.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]coding[\\/]matchxpath[\\/]InputMatchXpathForbidParameterizedConstructor.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]coding[\\/]matchxpath[\\/]InputMatchXpathForbidPrintStackTrace.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]coding[\\/]matchxpath[\\/]InputMatchXpathIllegalThrows.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]coding[\\/]matchxpath[\\/]InputMatchXpathIllegalThrows.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]coding[\\/]matchxpath[\\/]InputMatchXpathNoStackoverflowError.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]coding[\\/]matchxpath[\\/]InputMatchXpathSingleLineComments.java"/>
   <suppress id="UnnecessaryOkComment"
             files="checks[\\/]coding[\\/]modifiedcontrolvariable[\\/]InputModifiedControlVariableEnhancedForLoopVariable.java"/>
   <suppress id="UnnecessaryOkComment"

src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/matchxpath/InputMatchXpath.java

@@ -12,5 +12,5 @@ public void test() { }
 
     public void foo() { }
 
-    public void correct() { } // ok
+    public void correct() { }
 }

src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/matchxpath/InputMatchXpath2.java

@@ -12,5 +12,5 @@ public void test() { }
 
     public void foo() { }
 
-    public void correct() { } // ok
+    public void correct() { }
 }

src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/matchxpath/InputMatchXpath3.java

@@ -12,5 +12,5 @@ public void test() { } // violation
 
     public void foo() { } // violation
 
-    public void correct() { } // ok
+    public void correct() { }
 }

src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/matchxpath/InputMatchXpath4.java

@@ -12,5 +12,5 @@ public void test() { }
 
     public void foo() { }
 
-    public void correct() { } // ok
+    public void correct() { }
 }

src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/matchxpath/InputMatchXpathAvoidInstanceCreationWithoutVar.java

@@ -11,7 +11,7 @@
 public class InputMatchXpathAvoidInstanceCreationWithoutVar {
     public void test() {
         SomeObject a = new SomeObject(); // violation
-        var b = new SomeObject(); // OK
+        var b = new SomeObject();
     }
 
     class SomeObject {}

src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/matchxpath/InputMatchXpathExecutableStatementCount.java

@@ -10,7 +10,7 @@
 
 public class InputMatchXpathExecutableStatementCount {
     public void test() {
-        while (true) { // ok
+        while (true) {
             Runnable runnable = new Runnable() {
                 public void run() {
                     while (true) {

src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/matchxpath/InputMatchXpathForbidParameterizedConstructor.java

@@ -14,5 +14,5 @@ public InputMatchXpathForbidParameterizedConstructor(Object c) { } // violation
 
     public InputMatchXpathForbidParameterizedConstructor(int a, String b) { } // violation
 
-    public InputMatchXpathForbidParameterizedConstructor() { } // ok
+    public InputMatchXpathForbidParameterizedConstructor() { }
 }

src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/matchxpath/InputMatchXpathForbidPrintStackTrace.java

@@ -23,7 +23,6 @@ public void test2() {
       try {
          throw new IOException();
       } catch (IOException e) {
-         // ok
       }
    }
 }

src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/matchxpath/InputMatchXpathIllegalThrows.java

@@ -11,8 +11,8 @@
 
 public class InputMatchXpathIllegalThrows {
     public void func1() throws RuntimeException {} // violation
-    public void func2() throws Exception {}  // ok
+    public void func2() throws Exception {}
     public void func3() throws Error {}  // violation
     public void func4() throws Throwable {} // violation
-    public void func5() throws NullPointerException {} // ok
+    public void func5() throws NullPointerException {}
 }

src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/matchxpath/InputMatchXpathNoStackoverflowError.java

@@ -39,5 +39,5 @@ public class InputMatchXpathNoStackoverflowError {
       +""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""
       +""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""
       +""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""+""
-      ; // ok
+      ;
 }

src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/matchxpath/InputMatchXpathSingleLineComments.java

@@ -8,7 +8,7 @@
 
 public class InputMatchXpathSingleLineComments {
     void foo() {
-        int num; // Trailing comment // ok
+        int num; // Trailing comment
 
         boolean isTrue; //Some Comment // violation
         double pi = 3.14; //Constant PI value // violation