We can now use Java 8 streams to filter an iterator - more verbose.

johnmay · egonw · commit 314cd4b26937 · 2022-01-01T11:32:07.000+01:00
diff --git a/base/isomorphism/src/main/java/org/openscience/cdk/isomorphism/Mappings.java b/base/isomorphism/src/main/java/org/openscience/cdk/isomorphism/Mappings.java
@@ -27,7 +27,6 @@
 import com.google.common.collect.FluentIterable;
 import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.Iterables;
-import com.google.common.collect.Iterators;
 import org.openscience.cdk.graph.GraphUtil;
 import org.openscience.cdk.interfaces.IAtom;
 import org.openscience.cdk.interfaces.IAtomContainer;
@@ -40,6 +39,7 @@
 import java.util.Map;
 import java.util.function.Function;
 import java.util.function.Predicate;
+import java.util.stream.StreamSupport;
 
 /**
  * A fluent interface for handling (sub)-graph mappings from a query to a target
@@ -282,7 +282,8 @@ public Mappings uniqueAtoms() {
 
             @Override
             public Iterator<int[]> iterator() {
-                return Iterators.filter(iterable.iterator(), new UniqueAtomMatches()::test);
+                return StreamSupport.stream(iterable.spliterator(), false)
+                        .filter(new UniqueAtomMatches()).iterator();
             }
         });
     }
@@ -302,7 +303,8 @@ public Mappings uniqueBonds() {
 
             @Override
             public Iterator<int[]> iterator() {
-                return Iterators.filter(iterable.iterator(), new UniqueBondMatches(g)::test);
+                return StreamSupport.stream(iterable.spliterator(), false)
+                                    .filter(new UniqueBondMatches(g)).iterator();
             }
         });
     }
