Remove a redundant filter

QuiltMC · Jan 5, 2024 · 110f136 · 110f136
1 parent 6d20adb
commit 110f136
Showing 1 changed file with 4 additions and 10 deletions.
diff --git a/enigma/src/main/java/org/quiltmc/enigma/impl/translation/mapping/MappingsChecker.java b/enigma/src/main/java/org/quiltmc/enigma/impl/translation/mapping/MappingsChecker.java
@@ -7,17 +7,14 @@
 import org.quiltmc.enigma.api.translation.mapping.ResolutionStrategy;
 import org.quiltmc.enigma.api.translation.mapping.tree.EntryTree;
 import org.quiltmc.enigma.api.translation.mapping.tree.EntryTreeNode;
-import org.quiltmc.enigma.api.translation.representation.entry.ClassEntry;
 import org.quiltmc.enigma.api.translation.representation.entry.Entry;
-import org.quiltmc.enigma.api.translation.representation.entry.FieldEntry;
-import org.quiltmc.enigma.api.translation.representation.entry.LocalVariableEntry;
 import org.quiltmc.enigma.api.translation.representation.entry.MethodEntry;
 
 import java.util.Collection;
 import java.util.HashMap;
+import java.util.List;
 import java.util.Map;
 import java.util.function.BiConsumer;
-import java.util.stream.Stream;
 import java.util.stream.StreamSupport;
 
 public class MappingsChecker {
@@ -33,15 +30,12 @@ private Dropped dropMappings(ProgressListener progress, BiConsumer<Dropped, Entr
 		Dropped dropped = new Dropped();
 
 		// HashEntryTree#getAllEntries filters out empty classes
-		Stream<Entry<?>> allEntries = StreamSupport.stream(this.mappings.spliterator(), false).map(EntryTreeNode::getEntry);
-		Collection<Entry<?>> obfEntries = allEntries
-				.filter(e -> e instanceof ClassEntry || e instanceof MethodEntry || e instanceof FieldEntry || e instanceof LocalVariableEntry)
-				.toList();
+		List<? extends Entry<?>> entries = StreamSupport.stream(this.mappings.spliterator(), false).map(EntryTreeNode::getEntry).toList();
 
-		progress.init(obfEntries.size(), "Checking for dropped mappings");
+		progress.init(entries.size(), "Checking for dropped mappings");
 
 		int steps = 0;
-		for (Entry<?> entry : obfEntries) {
+		for (Entry<?> entry : entries) {
 			progress.step(steps++, entry.toString());
 			dropper.accept(dropped, entry);
 		}