Rewrite createLanguages() without Stream for simplicity and clearer stacktraces

Author: eregon

truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/LanguageCache.java

@@ -62,8 +62,6 @@
 import java.util.Set;
 import java.util.TreeSet;
 import java.util.function.Supplier;
-import java.util.stream.Stream;
-import java.util.stream.StreamSupport;
 
 import org.graalvm.home.HomeFinder;
 import org.graalvm.polyglot.SandboxPolicy;
@@ -273,7 +271,12 @@ private static synchronized Map<String, LanguageCache> createLanguages(List<Abst
             if (loader == null) {
                 continue;
             }
-            loadProviders(loader).filter((p) -> supplier.accepts(p.getClass())).forEach((p) -> loadLanguageImpl(p, caches, optionalResources));
+
+            for (TruffleLanguageProvider provider : ServiceLoader.load(TruffleLanguageProvider.class, loader)) {
+                if (supplier.accepts(provider.getClass())) {
+                    loadLanguageImpl(provider, caches, optionalResources);
+                }
+            }
         }
 
         Map<String, LanguageCache> idToCache = new LinkedHashMap<>();
@@ -299,10 +302,6 @@ private static synchronized Map<String, LanguageCache> createLanguages(List<Abst
         return idToCache;
     }
 
-    private static Stream<? extends TruffleLanguageProvider> loadProviders(ClassLoader loader) {
-        return StreamSupport.stream(ServiceLoader.load(TruffleLanguageProvider.class, loader).spliterator(), false);
-    }
-
     private static boolean hasSameCodeSource(LanguageCache first, LanguageCache second) {
         return first.provider.getClass() == second.provider.getClass();
     }