java.util.Iterable -> lang

PiperOrigin-RevId: 573238377
google · Oct 13, 2023 · b0f3199 · b0f3199
1 parent 1c62a56
commit b0f3199
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/core/src/main/java/com/google/errorprone/bugpatterns/threadsafety/HeldLockAnalyzer.java b/core/src/main/java/com/google/errorprone/bugpatterns/threadsafety/HeldLockAnalyzer.java
@@ -83,7 +83,7 @@ public final class HeldLockAnalyzer {
               .onDescendantOf("java.util.Map")
               .namedAnyOf("forEach", "replaceAll", "computeIfAbsent", "computeIfPresent", "merge"),
           instanceMethod().onDescendantOf("java.util.List").named("replaceAll"),
-          instanceMethod().onDescendantOf("java.util.Iterable").named("forEach"),
+          instanceMethod().onDescendantOf("java.lang.Iterable").named("forEach"),
           instanceMethod().onDescendantOf("java.util.Iterator").named("forEachRemaining"),
           staticMethod()
               .onClass("com.google.common.collect.Iterables")

diff --git a/core/src/test/java/com/google/errorprone/bugpatterns/threadsafety/GuardedByCheckerTest.java b/core/src/test/java/com/google/errorprone/bugpatterns/threadsafety/GuardedByCheckerTest.java
@@ -1963,4 +1963,20 @@ public void methodReference_referencedMethodIsFlagged() {
             "}")
         .doTest();
   }
+
+  @Test
+  public void lambdaMethodInvokedImmediately_shouldNotBeFlagged() {
+    compilationHelper
+        .addSourceLines(
+            "Test.java",
+            "import java.util.List;",
+            "import javax.annotation.concurrent.GuardedBy;",
+            "class Test {",
+            "  @GuardedBy(\"this\") private final Object o = new Object();",
+            "  public synchronized void test(List<?> xs) {",
+            "    xs.forEach(x -> o.toString());",
+            "  }",
+            "}")
+        .doTest();
+  }
 }