diff options
Diffstat (limited to 'src/main/java/cuchaz/enigma/mapping')
3 files changed, 29 insertions, 5 deletions
diff --git a/src/main/java/cuchaz/enigma/mapping/ClassMapping.java b/src/main/java/cuchaz/enigma/mapping/ClassMapping.java index 369ba8c..9c193ef 100644 --- a/src/main/java/cuchaz/enigma/mapping/ClassMapping.java +++ b/src/main/java/cuchaz/enigma/mapping/ClassMapping.java | |||
| @@ -577,7 +577,7 @@ public class ClassMapping implements Comparable<ClassMapping> { | |||
| 577 | } | 577 | } |
| 578 | 578 | ||
| 579 | // Used for tiny parsing to keep track of deobfuscate inner classes | 579 | // Used for tiny parsing to keep track of deobfuscate inner classes |
| 580 | public ClassMapping setDeobInner(String deobName) { | 580 | public ClassMapping setDeobfInner(String deobName) { |
| 581 | this.deobfFullName = deobName; | 581 | this.deobfFullName = deobName; |
| 582 | return this; | 582 | return this; |
| 583 | } | 583 | } |
| @@ -607,4 +607,21 @@ public class ClassMapping implements Comparable<ClassMapping> { | |||
| 607 | public boolean equals(Object obj) { | 607 | public boolean equals(Object obj) { |
| 608 | return obj instanceof ClassMapping && ((ClassMapping) obj).obfFullName.equals(this.obfFullName); | 608 | return obj instanceof ClassMapping && ((ClassMapping) obj).obfFullName.equals(this.obfFullName); |
| 609 | } | 609 | } |
| 610 | |||
| 611 | public boolean isEmpty() { | ||
| 612 | if (fieldsByDeobf.isEmpty() && methodsByDeobf.isEmpty() && deobfFullName == null && deobfName == null | ||
| 613 | && innerClassesByObfSimple.values().stream().allMatch(ClassMapping::isEmpty)) { | ||
| 614 | |||
| 615 | // check args | ||
| 616 | for (MethodMapping mapping : methodsByObf.values()) { | ||
| 617 | if (mapping.arguments().iterator().hasNext()) { | ||
| 618 | return false; | ||
| 619 | } | ||
| 620 | } | ||
| 621 | |||
| 622 | return true; | ||
| 623 | } | ||
| 624 | |||
| 625 | return false; | ||
| 626 | } | ||
| 610 | } | 627 | } |
diff --git a/src/main/java/cuchaz/enigma/mapping/MappingsEnigmaWriter.java b/src/main/java/cuchaz/enigma/mapping/MappingsEnigmaWriter.java index b29990f..e3302b1 100644 --- a/src/main/java/cuchaz/enigma/mapping/MappingsEnigmaWriter.java +++ b/src/main/java/cuchaz/enigma/mapping/MappingsEnigmaWriter.java | |||
| @@ -33,10 +33,19 @@ public class MappingsEnigmaWriter { | |||
| 33 | 33 | ||
| 34 | Mappings previousState = mappings.getPreviousState(); | 34 | Mappings previousState = mappings.getPreviousState(); |
| 35 | for (ClassMapping classMapping : sorted(mappings.classes())) { | 35 | for (ClassMapping classMapping : sorted(mappings.classes())) { |
| 36 | File result = new File(target, classMapping.getSaveName() + ".mapping"); | ||
| 37 | |||
| 36 | if (!classMapping.isDirty()) { | 38 | if (!classMapping.isDirty()) { |
| 37 | continue; | 39 | continue; |
| 38 | } | 40 | } |
| 39 | 41 | ||
| 42 | if (classMapping.isEmpty()) { | ||
| 43 | if (result.exists()) { | ||
| 44 | result.delete(); | ||
| 45 | } | ||
| 46 | continue; | ||
| 47 | } | ||
| 48 | |||
| 40 | if (previousState != null) { | 49 | if (previousState != null) { |
| 41 | ClassMapping previousClass = previousState.classesByObf.get(classMapping.getObfFullName()); | 50 | ClassMapping previousClass = previousState.classesByObf.get(classMapping.getObfFullName()); |
| 42 | File previousFile; | 51 | File previousFile; |
| @@ -50,8 +59,6 @@ public class MappingsEnigmaWriter { | |||
| 50 | } | 59 | } |
| 51 | } | 60 | } |
| 52 | 61 | ||
| 53 | File result = new File(target, classMapping.getSaveName() + ".mapping"); | ||
| 54 | |||
| 55 | File packageFile = result.getParentFile(); | 62 | File packageFile = result.getParentFile(); |
| 56 | if (!packageFile.exists()) { | 63 | if (!packageFile.exists()) { |
| 57 | packageFile.mkdirs(); | 64 | packageFile.mkdirs(); |
diff --git a/src/main/java/cuchaz/enigma/mapping/MappingsTinyReader.java b/src/main/java/cuchaz/enigma/mapping/MappingsTinyReader.java index 69d5684..756ac43 100644 --- a/src/main/java/cuchaz/enigma/mapping/MappingsTinyReader.java +++ b/src/main/java/cuchaz/enigma/mapping/MappingsTinyReader.java | |||
| @@ -17,7 +17,7 @@ public class MappingsTinyReader { | |||
| 17 | public ClassMapping readClass(String[] parts) { | 17 | public ClassMapping readClass(String[] parts) { |
| 18 | // Extract the inner naming of the deob form if it have one | 18 | // Extract the inner naming of the deob form if it have one |
| 19 | String deobName = parts[2].contains("$") ? parts[2].substring(parts[2].lastIndexOf('$') + 1) : parts[2]; | 19 | String deobName = parts[2].contains("$") ? parts[2].substring(parts[2].lastIndexOf('$') + 1) : parts[2]; |
| 20 | return new ClassMapping(parts[1], deobName).setDeobInner(parts[2]); | 20 | return new ClassMapping(parts[1], deobName).setDeobfInner(parts[2]); |
| 21 | } | 21 | } |
| 22 | 22 | ||
| 23 | public FieldMapping readField(String[] parts) { | 23 | public FieldMapping readField(String[] parts) { |
| @@ -52,7 +52,7 @@ public class MappingsTinyReader { | |||
| 52 | parts[2]; | 52 | parts[2]; |
| 53 | 53 | ||
| 54 | // Add full deob name to the class mapping to handle inner class after this loop | 54 | // Add full deob name to the class mapping to handle inner class after this loop |
| 55 | classMappingMap.put(parts[2], classMapping.setDeobInner(parts[2])); | 55 | classMappingMap.put(parts[2], classMapping.setDeobfInner(parts[2])); |
| 56 | classMapping.setDeobfName(deobName); | 56 | classMapping.setDeobfName(deobName); |
| 57 | 57 | ||
| 58 | // Avoid to make the mapping dirty directly at the startup | 58 | // Avoid to make the mapping dirty directly at the startup |