Use removeIf() because why not. (#5)

PaintNinja · web-flow · commit 4af4e575252f · 2024-05-30T13:20:10.000-07:00
diff --git a/bs-dev/src/main/java/net/minecraftforge/bootstrap/dev/BootstrapDevClasspathFixer.java b/bs-dev/src/main/java/net/minecraftforge/bootstrap/dev/BootstrapDevClasspathFixer.java
@@ -88,10 +88,13 @@ private boolean processExplicit(List<Path[]> classpath) {
         }
 
         // Remove any entries that are only a single path, no need to merge them
+        map.values().removeIf(paths -> paths.size() <= 1);
+        /*
         for (var itr  = map.values().iterator(); itr.hasNext(); ) {
             if (itr.next().size() <= 1)
                 itr.remove();
         }
+        */
 
         // No explicit paths set, so nope out
         if (map.isEmpty())

Original file line number	Diff line number	Diff line change
`@@ -88,10 +88,13 @@ private boolean processExplicit(List<Path[]> classpath) {`
`88`	`88`	`}`
`89`	`89`
`90`	`90`	`// Remove any entries that are only a single path, no need to merge them`
	`91`	`+ map.values().removeIf(paths -> paths.size() <= 1);`
	`92`	`+ /*`
`91`	`93`	`for (var itr = map.values().iterator(); itr.hasNext(); ) {`
`92`	`94`	`if (itr.next().size() <= 1)`
`93`	`95`	`itr.remove();`
`94`	`96`	`}`
	`97`	`+ */`
`95`	`98`
`96`	`99`	`// No explicit paths set, so nope out`
`97`	`100`	`if (map.isEmpty())`