Resolve all failed tests

4 files changed, 10 insertions, 13 deletions

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();
         }