comments

uber · Mar 27, 2024 · 3d985b7 · 3d985b7
1 parent 24b0275
commit 3d985b7
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/nullaway/src/main/java/com/uber/nullaway/handlers/StreamNullabilityPropagator.java b/nullaway/src/main/java/com/uber/nullaway/handlers/StreamNullabilityPropagator.java
@@ -254,6 +254,11 @@ public void onMatchMethodInvocation(
           }
         } else {
           if (methodSymbol.getParameters().length() == 1) {
+            // We can have multiple CollectLikeMethodRecords for a single collect method, reflecting
+            // the different possible collector factory methods whose result may be passed to a
+            // collect call.  At a single collect call site, at most one of these records will be
+            // relevant. So, we loop through them all, but break out of the loop as soon as we find
+            // one that matches.
             for (CollectLikeMethodRecord collectlikeMethodRecord :
                 streamType.getCollectlikeMethodRecords(methodSymbol)) {
               boolean handled = handleCollectCall(tree, collectlikeMethodRecord);