Skip to content

Commit

Permalink
Issue checkstyle#14625: move TailRecursion inspection problems to sep…
Browse files Browse the repository at this point in the history
…arate issue
  • Loading branch information
MANISH-K-07 committed Apr 20, 2024
1 parent bb56e9d commit ce0e4ed
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 8 deletions.
Expand Up @@ -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) {
Expand Down
Expand Up @@ -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) {
Expand Down
Expand Up @@ -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)
Expand All @@ -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)
Expand Down
Expand Up @@ -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")
Expand Down
Expand Up @@ -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) {
Expand Down
Expand Up @@ -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++;
Expand Down

0 comments on commit ce0e4ed

Please sign in to comment.