From 69f2e98049c852680730daf12099771ecae0e489 Mon Sep 17 00:00:00 2001
From: Tara <keshwarithara64@gmail.com>
Date: Sat, 23 Mar 2024 11:15:57 +0530
Subject: [PATCH] Issue #13213: Remove '//ok' comments from Input files

---
 config/checkstyle-input-suppressions.xml      | 34 -------------------
 .../InputJavadocMethodSetterGetter.java       | 34 +++++++++----------
 2 files changed, 17 insertions(+), 51 deletions(-)

diff --git a/config/checkstyle-input-suppressions.xml b/config/checkstyle-input-suppressions.xml
index e3c3361ebe6..2129ae8f60f 100644
--- a/config/checkstyle-input-suppressions.xml
+++ b/config/checkstyle-input-suppressions.xml
@@ -431,40 +431,6 @@
             files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodScopeInnerInterfaces.java"/>
   <suppress id="UnnecessaryOkComment"
             files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodScopeInnerInterfaces.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
-  <suppress id="UnnecessaryOkComment"
-            files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSetterGetter.java"/>
   <suppress id="UnnecessaryOkComment"
             files="checks[\\/]javadoc[\\/]javadocmethod[\\/]InputJavadocMethodSurroundingAccessModifier.java"/>
   <suppress id="UnnecessaryOkComment"
diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/checks/javadoc/javadocmethod/InputJavadocMethodSetterGetter.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/javadoc/javadocmethod/InputJavadocMethodSetterGetter.java
index 92394e972bd..fdfe719ecd0 100644
--- a/src/test/resources/com/puppycrawl/tools/checkstyle/checks/javadoc/javadocmethod/InputJavadocMethodSetterGetter.java
+++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/javadoc/javadocmethod/InputJavadocMethodSetterGetter.java
@@ -14,75 +14,75 @@
 
 public class InputJavadocMethodSetterGetter
 {
-    private int mNumber; // ok
+    private int mNumber;
 
-    public void setNumber(final int number) // ok
+    public void setNumber(final int number)
     {
         mNumber = number;
     }
 
-    public int getNumber() // ok
+    public int getNumber()
     {
         return mNumber;
     }
 
-    public void setNumber1() // ok
+    public void setNumber1()
     {
         mNumber = mNumber;
     }
 
-    public void setNumber2(int number) // ok
+    public void setNumber2(int number)
     {
         mNumber = number;
         firePropertyChanged();
     }
 
-    public void getNumber2() // ok
+    public void getNumber2()
     {
     }
 
-    public int getCost1(int forMe) // ok
+    public int getCost1(int forMe)
     {
         return 666;
     }
 
-    public int getCost2() // ok
+    public int getCost2()
     {
             mNumber = 4;
         return 666;
     }
 
-    public int getCost3() throws Exception // ok
+    public int getCost3() throws Exception
     {
         return 666;
     }
 
-    public boolean isSomething() // ok
+    public boolean isSomething()
     {
         return false;
     }
 
-    private void firePropertyChanged(){} // ok
+    private void firePropertyChanged(){}
 
-    Object setObject(Object object) { // ok
+    Object setObject(Object object) {
         return new Object();
     }
 
-    Object getNext() { // ok
+    Object getNext() {
         throw new UnsupportedOperationException();
     }
 
-    public void setWithoutAssignment(Object object) { // ok
+    public void setWithoutAssignment(Object object) {
         object.notify();
     }
 
-    InputJavadocMethodSetterGetter() {} // ok
+    InputJavadocMethodSetterGetter() {}
 
-    public InputJavadocMethodSetterGetter(Object object) throws Exception {} // ok
+    public InputJavadocMethodSetterGetter(Object object) throws Exception {}
 
 }
 
-interface TestInterface { // ok
+interface TestInterface {
     void setObject(Object object);
 
     Object getObject();