diff options
Diffstat (limited to 'src/main/java')
| -rw-r--r-- | src/main/java/cuchaz/enigma/mapping/FieldMapping.java | 18 | ||||
| -rw-r--r-- | src/main/java/cuchaz/enigma/mapping/MethodMapping.java | 15 |
2 files changed, 30 insertions, 3 deletions
diff --git a/src/main/java/cuchaz/enigma/mapping/FieldMapping.java b/src/main/java/cuchaz/enigma/mapping/FieldMapping.java index e75485c..83e2277 100644 --- a/src/main/java/cuchaz/enigma/mapping/FieldMapping.java +++ b/src/main/java/cuchaz/enigma/mapping/FieldMapping.java | |||
| @@ -10,6 +10,8 @@ | |||
| 10 | ******************************************************************************/ | 10 | ******************************************************************************/ |
| 11 | package cuchaz.enigma.mapping; | 11 | package cuchaz.enigma.mapping; |
| 12 | 12 | ||
| 13 | import cuchaz.enigma.throwables.IllegalNameException; | ||
| 14 | |||
| 13 | public class FieldMapping implements Comparable<FieldMapping>, MemberMapping<FieldEntry> { | 15 | public class FieldMapping implements Comparable<FieldMapping>, MemberMapping<FieldEntry> { |
| 14 | 16 | ||
| 15 | private String obfName; | 17 | private String obfName; |
| @@ -49,8 +51,20 @@ public class FieldMapping implements Comparable<FieldMapping>, MemberMapping<Fie | |||
| 49 | this.deobfName = NameValidator.validateFieldName(val); | 51 | this.deobfName = NameValidator.validateFieldName(val); |
| 50 | } | 52 | } |
| 51 | 53 | ||
| 52 | public void setObfName(String val) { | 54 | public void setObfName(String name) { |
| 53 | this.obfName = NameValidator.validateFieldName(val); | 55 | try |
| 56 | { | ||
| 57 | NameValidator.validateMethodName(name); | ||
| 58 | } catch (IllegalNameException ex) | ||
| 59 | { | ||
| 60 | // Invalid name, damn obfuscation! Map to a deob name with another name to avoid issues | ||
| 61 | if (this.deobfName == null) | ||
| 62 | { | ||
| 63 | System.err.println("WARNING: " + name + " is conflicting, auto deobfuscate to " + (name + "_auto_deob")); | ||
| 64 | setDeobfName(name + "_auto_deob"); | ||
| 65 | } | ||
| 66 | } | ||
| 67 | this.obfName = name; | ||
| 54 | } | 68 | } |
| 55 | 69 | ||
| 56 | public Type getObfType() { | 70 | public Type getObfType() { |
diff --git a/src/main/java/cuchaz/enigma/mapping/MethodMapping.java b/src/main/java/cuchaz/enigma/mapping/MethodMapping.java index 455ff6b..f973d6b 100644 --- a/src/main/java/cuchaz/enigma/mapping/MethodMapping.java +++ b/src/main/java/cuchaz/enigma/mapping/MethodMapping.java | |||
| @@ -14,6 +14,7 @@ import com.google.common.collect.Maps; | |||
| 14 | 14 | ||
| 15 | import java.util.Map; | 15 | import java.util.Map; |
| 16 | 16 | ||
| 17 | import cuchaz.enigma.throwables.IllegalNameException; | ||
| 17 | import cuchaz.enigma.throwables.MappingConflict; | 18 | import cuchaz.enigma.throwables.MappingConflict; |
| 18 | 19 | ||
| 19 | public class MethodMapping implements Comparable<MethodMapping>, MemberMapping<BehaviorEntry> { | 20 | public class MethodMapping implements Comparable<MethodMapping>, MemberMapping<BehaviorEntry> { |
| @@ -75,7 +76,19 @@ public class MethodMapping implements Comparable<MethodMapping>, MemberMapping<B | |||
| 75 | } | 76 | } |
| 76 | 77 | ||
| 77 | public void setObfName(String name) { | 78 | public void setObfName(String name) { |
| 78 | this.obfName = NameValidator.validateMethodName(name); | 79 | try |
| 80 | { | ||
| 81 | NameValidator.validateMethodName(name); | ||
| 82 | } catch (IllegalNameException ex) | ||
| 83 | { | ||
| 84 | // Invalid name, damn obfuscation! Map to a deob name with another name to avoid issues | ||
| 85 | if (this.deobfName == null) | ||
| 86 | { | ||
| 87 | System.err.println("WARNING: " + name + " is conflicting, auto deobfuscate to " + (name + "_auto_deob")); | ||
| 88 | setDeobfName(name + "_auto_deob"); | ||
| 89 | } | ||
| 90 | } | ||
| 91 | this.obfName = name; | ||
| 79 | } | 92 | } |
| 80 | 93 | ||
| 81 | public void setObfSignature(Signature val) { | 94 | public void setObfSignature(Signature val) { |