Same construct above as well

timtebeek · timtebeek · commit 960e6013d9af · 2026-02-13T13:52:14.000+01:00
diff --git a/rewrite-core/src/main/java/org/openrewrite/internal/ListUtils.java b/rewrite-core/src/main/java/org/openrewrite/internal/ListUtils.java
@@ -215,8 +215,7 @@ public static <T> List<T> insert(@Nullable List<T> ls, @Nullable T t, int index)
         }
 
         if (newLs != ls && nullEncountered) {
-            //noinspection StatementWithEmptyBody
-            while (newLs.remove(null)) ;
+            newLs.removeIf(Objects::isNull);
         }
 
         //noinspection NullableProblems

Original file line number	Diff line number	Diff line change
`@@ -215,8 +215,7 @@ public static <T> List<T> insert(@Nullable List<T> ls, @Nullable T t, int index)`
`215`	`215`	`}`
`216`	`216`
`217`	`217`	`if (newLs != ls && nullEncountered) {`
`218`		`- //noinspection StatementWithEmptyBody`
`219`		`- while (newLs.remove(null)) ;`
	`218`	`+ newLs.removeIf(Objects::isNull);`
`220`	`219`	`}`
`221`	`220`
`222`	`221`	`//noinspection NullableProblems`