Correct behaviour of removeRedundantExclusions

src/main/java/me/nallar/modpatcher/LaunchClassLoaderUtil.java

@@ -293,15 +293,16 @@ public static void removeRedundantExclusions() {
 		removeRedundantExclusions(getClassLoaderExceptions());
 	}
 
-	private static void removeRedundantExclusions(Set<String> transformerExceptions) {
-		Iterator<String> parts = transformerExceptions.iterator();
-		while (parts.hasNext()) {
-			String part = parts.next();
-
-			for (String part2 : new HashSet<>(transformerExceptions)) {
-				if (!part.equals(part2) && part.startsWith(part2)) {
-					parts.remove();
-				}
+	static void removeRedundantExclusions(Set<String> transformerExceptions) {
+		HashSet<String> old = new HashSet<>(transformerExceptions);
+
+		for (String exclusion : old) {
+			for (String exclusion2 : old) {
+				if (exclusion.equals(exclusion2))
+					continue;
+
+				if (exclusion.startsWith(exclusion2))
+					transformerExceptions.remove(exclusion);
 			}
 		}
 	}

src/test/java/me/nallar/modpatcher/LaunchClassLoaderUtilTest.java

@@ -0,0 +1,28 @@
+package me.nallar.modpatcher;
+
+import org.junit.Test;
+
+import java.util.*;
+
+public class LaunchClassLoaderUtilTest {
+	@Test
+	public void TestRemoveRedundantExclusions() {
+		HashSet<String> test = new HashSet<>();
+		test.add("me.nallar.test");
+		test.add("me.");
+		test.add("me.nallar");
+		test.add("org.example.banana");
+		test.add("org.example.apple");
+		test.add("org.example.pear");
+		test.add("org.example.apple.pear");
+
+		LaunchClassLoaderUtil.removeRedundantExclusions(test);
+		assert test.contains("me.");
+		assert test.contains("org.example.banana");
+		assert test.contains("org.example.apple");
+		assert test.contains("org.example.pear");
+		assert !test.contains("me.nallar");
+		assert !test.contains("me.nallar.test");
+		assert !test.contains("org.example.apple.pear");
+	}
+}