Issue #14625: move TailRecursion inspection problems to separate issue

MANISH-K-07 · nrmancuso · commit 508757d61f58 · 2024-04-26T22:18:21.000-04:00
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/api/FullIdent.java b/src/main/java/com/puppycrawl/tools/checkstyle/api/FullIdent.java
@@ -75,7 +75,7 @@ public static FullIdent createFullIdent(DetailAST ast) {
      * @param full the FullIdent to add to
      * @param ast the node to recurse from
      * @noinspection TailRecursion
-     * @noinspectionreason TailRecursion - until issue #14625
+     * @noinspectionreason TailRecursion - until issue #14814
      */
     private static void extractFullIdent(FullIdent full, DetailAST ast) {
         if (ast != null) {
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/annotation/SuppressWarningsCheck.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/annotation/SuppressWarningsCheck.java
@@ -310,7 +310,7 @@ private static String removeQuotes(final String warning) {
      * @param cond a Conditional type
      *     {@link TokenTypes#QUESTION QUESTION}
      * @noinspection TailRecursion
-     * @noinspectionreason TailRecursion - until issue #14625
+     * @noinspectionreason TailRecursion - until issue #14814
      */
     private void walkConditional(final DetailAST cond) {
         if (cond.getType() == TokenTypes.QUESTION) {
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/coding/HiddenFieldCheck.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/coding/HiddenFieldCheck.java
@@ -650,7 +650,7 @@ public void addStaticField(String field) {
          * @param field the field to check
          * @return true if this FieldFrame contains instance field
          * @noinspection TailRecursion
-         * @noinspectionreason TailRecursion - until issue #14625
+         * @noinspectionreason TailRecursion - until issue #14814
          */
         public boolean containsInstanceField(String field) {
             return instanceFields.contains(field)
@@ -664,7 +664,7 @@ public boolean containsInstanceField(String field) {
          * @param field the field to check
          * @return true if this FieldFrame contains static field
          * @noinspection TailRecursion
-         * @noinspectionreason TailRecursion - until issue #14625
+         * @noinspectionreason TailRecursion - until issue #14814
          */
         public boolean containsStaticField(String field) {
             return staticFields.contains(field)
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/regexp/RegexpCheck.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/regexp/RegexpCheck.java
@@ -306,7 +306,7 @@ public void beginTree(DetailAST rootAST) {
      * Recursive method that finds the matches.
      *
      * @noinspection TailRecursion
-     * @noinspectionreason TailRecursion - until issue #14625
+     * @noinspectionreason TailRecursion - until issue #14814
      */
     // suppress deprecation until https://github.com/checkstyle/checkstyle/issues/11166
     @SuppressWarnings("deprecation")
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/regexp/SinglelineDetector.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/regexp/SinglelineDetector.java
@@ -98,7 +98,7 @@ private void resetState() {
      * @param matcher the matcher to use
      * @param startPosition the position to start searching from.
      * @noinspection TailRecursion
-     * @noinspectionreason TailRecursion - until issue #14625
+     * @noinspectionreason TailRecursion - until issue #14814
      */
     private void checkLine(int lineNo, String line, Matcher matcher,
             int startPosition) {
diff --git a/src/test/java/com/puppycrawl/tools/checkstyle/utils/ModuleReflectionUtilTest.java b/src/test/java/com/puppycrawl/tools/checkstyle/utils/ModuleReflectionUtilTest.java
@@ -385,11 +385,11 @@ protected InvalidNonDefaultConstructorClass(int data) {
         }
 
         /**
-         * Temporary java doc.
+         * Method has tail recursive call.
          *
          * @param data of int type.
          * @noinspection TailRecursion
-         * @noinspectionreason TailRecursion - until issue #14625
+         * @noinspectionreason TailRecursion - until issue #14814
          */
         public final void method(int data) {
             field++;

Original file line number	Diff line number	Diff line change
`@@ -306,7 +306,7 @@ public void beginTree(DetailAST rootAST) {`
`306`	`306`	`* Recursive method that finds the matches.`
`307`	`307`	`*`
`308`	`308`	`* @noinspection TailRecursion`
`309`		`- * @noinspectionreason TailRecursion - until issue #14625`
	`309`	`+ * @noinspectionreason TailRecursion - until issue #14814`
`310`	`310`	`*/`
`311`	`311`	`// suppress deprecation until https://github.com/checkstyle/checkstyle/issues/11166`
`312`	`312`	`@SuppressWarnings("deprecation")`
Original file line number	Diff line number	Diff line change
`@@ -385,11 +385,11 @@ protected InvalidNonDefaultConstructorClass(int data) {`
`385`	`385`	`}`
`386`	`386`
`387`	`387`	`/**`
`388`		`- * Temporary java doc.`
	`388`	`+ * Method has tail recursive call.`
`389`	`389`	`*`
`390`	`390`	`* @param data of int type.`
`391`	`391`	`* @noinspection TailRecursion`
`392`		`- * @noinspectionreason TailRecursion - until issue #14625`
	`392`	`+ * @noinspectionreason TailRecursion - until issue #14814`
`393`	`393`	`*/`
`394`	`394`	`public final void method(int data) {`
`395`	`395`	`field++;`