Don't fire CanIgnoreReturnValueSuggester for simple return param;…

… implementations. #checkreturnvalue PiperOrigin-RevId: 667620072
google · Aug 26, 2024 · 1569624 · 1569624
1 parent 0a5a5b8
commit 1569624
Show file tree

Hide file tree

Showing 2 changed files with 44 additions and 0 deletions.
diff --git a/...ava/com/google/errorprone/bugpatterns/checkreturnvalue/CanIgnoreReturnValueSuggester.java b/...ava/com/google/errorprone/bugpatterns/checkreturnvalue/CanIgnoreReturnValueSuggester.java
@@ -153,6 +153,16 @@ public Description matchMethod(MethodTree methodTree, VisitorState state) {
 
     // if the method always return a single input param (of the same type), make it CIRV
     if (methodAlwaysReturnsInputParam(methodTree, state)) {
+      // if the method _only_ returns an input param, bail out
+      if (methodTree.getBody() != null && methodTree.getBody().getStatements().size() == 1) {
+        StatementTree onlyStatement = methodTree.getBody().getStatements().get(0);
+        if (onlyStatement instanceof ReturnTree) {
+          ReturnTree returnTree = (ReturnTree) onlyStatement;
+          if (returnTree.getExpression() instanceof IdentifierTree) {
+            return Description.NO_MATCH;
+          }
+        }
+      }
       return annotateWithCanIgnoreReturnValue(methodTree, state);
     }
 

diff --git a/...com/google/errorprone/bugpatterns/checkreturnvalue/CanIgnoreReturnValueSuggesterTest.java b/...com/google/errorprone/bugpatterns/checkreturnvalue/CanIgnoreReturnValueSuggesterTest.java
@@ -56,6 +56,36 @@ public void simpleCase() {
         .doTest();
   }
 
+  @Test
+  public void b362106953_returnThis() {
+    helper
+        .addInputLines(
+            "Client.java",
+            "package com.google.frobber;",
+            "public final class Client {",
+            "  public Client setName(String name) {",
+            "    return this;",
+            "  }",
+            "}")
+        .expectUnchanged()
+        .doTest();
+  }
+
+  @Test
+  public void b362106953_returnParam() {
+    helper
+        .addInputLines(
+            "Client.java",
+            "package com.google.frobber;",
+            "public final class Client {",
+            "  public String setName(String name) {",
+            "    return name;",
+            "  }",
+            "}")
+        .expectUnchanged()
+        .doTest();
+  }
+
   @Test
   public void parenthesizedCastThis() {
     helper
@@ -92,6 +122,7 @@ public void returnsInputParam() {
             "package com.google.frobber;",
             "public final class Client {",
             "  public String method(String name) {",
+            "    System.out.println(name);",
             "    return name;",
             "  }",
             "}")
@@ -102,6 +133,7 @@ public void returnsInputParam() {
             "public final class Client {",
             "  @CanIgnoreReturnValue",
             "  public String method(String name) {",
+            "    System.out.println(name);",
             "    return name;",
             "  }",
             "}")
@@ -157,6 +189,7 @@ public void returnInputParams_multipleParams() {
             "package com.google.frobber;",
             "public final class ReturnInputParam {",
             "  public static StringBuilder append(StringBuilder input, String name) {",
+            "    input.append(name);",
             "    return input;",
             "  }",
             "}")
@@ -167,6 +200,7 @@ public void returnInputParams_multipleParams() {
             "public final class ReturnInputParam {",
             "  @CanIgnoreReturnValue",
             "  public static StringBuilder append(StringBuilder input, String name) {",
+            "    input.append(name);",
             "    return input;",
             "  }",
             "}")