From 7ec433f1ecc5b44f8b690c7443d5e38ac0e6422d Mon Sep 17 00:00:00 2001
From: gegy1000
Date: Fri, 22 Jun 2018 22:29:41 +0200
Subject: Resolve all failed tests

---
 src/main/java/cuchaz/enigma/Deobfuscator.java            | 16 +++++++---------
 src/main/java/cuchaz/enigma/analysis/JarIndex.java       |  3 +--
 .../java/cuchaz/enigma/analysis/TranslationIndex.java    |  2 +-
 src/main/java/cuchaz/enigma/gui/GuiController.java       |  2 +-
 4 files changed, 10 insertions(+), 13 deletions(-)

(limited to 'src/main/java/cuchaz/enigma')

diff --git a/src/main/java/cuchaz/enigma/Deobfuscator.java b/src/main/java/cuchaz/enigma/Deobfuscator.java
index 6bbfd07..bdd8873 100644
--- a/src/main/java/cuchaz/enigma/Deobfuscator.java
+++ b/src/main/java/cuchaz/enigma/Deobfuscator.java
@@ -46,7 +46,6 @@ import java.util.jar.JarOutputStream;
 public class Deobfuscator {
 
 	private final ReferencedEntryPool entryPool = new ReferencedEntryPool();
-	private final JarFile jar;
 	private final ParsedJar parsedJar;
 	private final DecompilerSettings settings;
 	private final JarIndex jarIndex;
@@ -54,9 +53,8 @@ public class Deobfuscator {
 	private final Map<TranslationDirection, Translator> translatorCache;
 	private Mappings mappings;
 
-	public Deobfuscator(JarFile jar) throws IOException {
-		this.jar = jar;
-		this.parsedJar = new ParsedJar(jar);
+	public Deobfuscator(ParsedJar jar) {
+		this.parsedJar = jar;
 
 		// build the jar index
 		this.jarIndex = new JarIndex(entryPool);
@@ -79,12 +77,12 @@ public class Deobfuscator {
 		setMappings(new Mappings());
 	}
 
-	public ParsedJar getJar() {
-		return this.parsedJar;
+	public Deobfuscator(JarFile jar) throws IOException {
+		this(new ParsedJar(jar));
 	}
 
-	public String getJarName() {
-		return this.jar.getName();
+	public ParsedJar getJar() {
+		return this.parsedJar;
 	}
 
 	public JarIndex getJarIndex() {
@@ -388,7 +386,7 @@ public class Deobfuscator {
 
 		classMapping.markDirty();
 		renameClassMap.put(classMapping, renameEntries);
-		for(ClassMapping innerClass : classMapping.innerClasses()){
+		for (ClassMapping innerClass : classMapping.innerClasses()) {
 			rebuildMethodNames(innerClass, renameClassMap);
 		}
 	}
diff --git a/src/main/java/cuchaz/enigma/analysis/JarIndex.java b/src/main/java/cuchaz/enigma/analysis/JarIndex.java
index 87e6e88..8172dea 100644
--- a/src/main/java/cuchaz/enigma/analysis/JarIndex.java
+++ b/src/main/java/cuchaz/enigma/analysis/JarIndex.java
@@ -148,8 +148,7 @@ public class JarIndex {
 
 	public void indexInnerClass(ClassEntry innerEntry, ClassEntry outerEntry) {
 		this.innerClassesByOuter.put(outerEntry, innerEntry);
-		boolean innerWasAdded = this.outerClassesByInner.put(innerEntry, outerEntry) == null;
-		assert (innerWasAdded);
+		this.outerClassesByInner.putIfAbsent(innerEntry, outerEntry);
 	}
 
 	private MethodEntry findBridgedMethod(MethodDefEntry method) {
diff --git a/src/main/java/cuchaz/enigma/analysis/TranslationIndex.java b/src/main/java/cuchaz/enigma/analysis/TranslationIndex.java
index 644a3f2..b2ddc5f 100644
--- a/src/main/java/cuchaz/enigma/analysis/TranslationIndex.java
+++ b/src/main/java/cuchaz/enigma/analysis/TranslationIndex.java
@@ -84,7 +84,7 @@ public class TranslationIndex {
 
 		// add the superclass
 		ClassEntry superclassEntry = entryPool.getClass(superName);
-		if (!isJre(superclassEntry)) {
+		if (superclassEntry != null) {
 			this.superclasses.put(classEntry, superclassEntry);
 		}
 
diff --git a/src/main/java/cuchaz/enigma/gui/GuiController.java b/src/main/java/cuchaz/enigma/gui/GuiController.java
index 837cff2..ae1b652 100644
--- a/src/main/java/cuchaz/enigma/gui/GuiController.java
+++ b/src/main/java/cuchaz/enigma/gui/GuiController.java
@@ -58,7 +58,7 @@ public class GuiController {
 	public void openJar(final JarFile jar) throws IOException {
 		this.gui.onStartOpenJar();
 		this.deobfuscator = new Deobfuscator(jar);
-		this.gui.onFinishOpenJar(this.deobfuscator.getJarName());
+		this.gui.onFinishOpenJar(jar.getName());
 		refreshClasses();
 	}
 
-- 
cgit v1.2.3