I guess when I rebuilt the test jar the order of the classes changed, 1->100 not 1->2, which made the class inheritance order inconsistent

src/main/java/net/runelite/deob/ClassGroup.java

@@ -3,6 +3,7 @@ package net.runelite.deob;
 import java.io.DataInputStream;
 import java.io.IOException;
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 import net.runelite.deob.attributes.Code;
 
@@ -47,9 +48,32 @@ public class ClassGroup
 	
 	public void initialize()
 	{
+		sort();
 		buildClassGraph();
 		lookup();
 	}
+	
+	private int classNum(ClassFile cf)
+	{
+		if (cf.getName().startsWith("class"))
+			return Integer.parseInt(cf.getName().substring(5));
+		
+		return -1;
+	}
+	
+	// order of classes affects class graph (eg order of children classes) which affects comparing two classgroups
+	private void sort()
+	{
+		// only sort renamed jars
+		for (ClassFile c : classes)
+			if (c.getName().startsWith("class") == false && !c.getName().equals("client"))
+				return;
+		
+		Collections.sort(classes, (c1, c2) -> {
+			int n1 = classNum(c1), n2 = classNum(c2);
+			return Integer.compare(n1, n2);
+		});
+	}
 
 	public void buildClassGraph()
 	{

src/test/java/net/runelite/deob/deobfuscators/rename/MapStaticTest.java

@@ -2,6 +2,9 @@ package net.runelite.deob.deobfuscators.rename;
 
 import java.io.File;
 import java.io.IOException;
+import java.util.ArrayList;
+import java.util.List;
+import net.runelite.deob.ClassFile;
 import net.runelite.deob.ClassGroup;
 import net.runelite.deob.Method;
 import net.runelite.deob.util.JarUtil;
@@ -68,4 +71,57 @@ public class MapStaticTest
 		
 		ParallelExecutorMapping mappings = MappingExecutorUtil.map(m1, m2);
 	}
+	
+	//@Test
+	public void test2() throws Exception
+	{
+		ClassGroup group1 = JarUtil.loadJar(new File("d:/rs/07/adamin1.jar"));
+		ClassGroup group2 = JarUtil.loadJar(new File("d:/rs/07/adamin2.jar"));
+		
+		List<ClassFile> cl1 = group1.getClasses(),
+		cl2 = group2.getClasses();
+		
+		for (int i = 0; i < 200; ++i)
+		{
+			ClassFile c1 = cl1.get(i), c2 = cl2.get(i);
+			System.out.println(c1 + " <-> " + c2);
+		}
+		//Assert.assertEquals(cl1.size(), cl2.size());
+		
+		Method m1 = group1.findClass("class66").findMethod("vmethod3787");
+		Method m2 = group2.findClass("class66").findMethod("vmethod3664");
+		
+		List<Method> list = new ArrayList<>();
+		findMethodFromClass(m1.getPoolMethod(), list, group1.findClass("class66"));
+		
+		System.out.println("break");
+		
+		List<Method> list2 = new ArrayList<>();
+		findMethodFromClass(m2.getPoolMethod(), list2, group2.findClass("class66"));
+		
+		Assert.assertEquals(list.size(), list2.size());
+		
+		for (int i = 0; i < list.size(); ++i)
+		{
+			m1 = list.get(i);
+			m2 = list2.get(i);
+			
+			System.out.println(m1 + " vs " + m2);
+
+			Assert.assertEquals(m1.getMethods().getClassFile().getName(), m2.getMethods().getClassFile().getName());
+		}
+	}
+	
+	private void findMethodFromClass(net.runelite.deob.pool.Method method, List<Method> list, ClassFile clazz)
+	{
+		net.runelite.deob.Method m = clazz.findMethodDeep(method.getNameAndType());
+		if (m != null && !list.contains(m))
+			list.add(m);
+	
+		for (ClassFile cf : clazz.getChildren())
+		{
+			System.out.println("Child of " + clazz + ": " +cf);
+			findMethodFromClass(method, list, cf);
+		}
+	}
 }