From 59e189bef2b5e6d129fb7c2c988ed0b2130e36ac Mon Sep 17 00:00:00 2001 From: lclc98 Date: Mon, 4 Jul 2016 18:14:22 +1000 Subject: Reformat --- .../java/cuchaz/enigma/mapping/ClassMapping.java | 46 ++++++---------------- 1 file changed, 12 insertions(+), 34 deletions(-) (limited to 'src/main/java/cuchaz/enigma/mapping/ClassMapping.java') diff --git a/src/main/java/cuchaz/enigma/mapping/ClassMapping.java b/src/main/java/cuchaz/enigma/mapping/ClassMapping.java index f831a3b..b2c076a 100644 --- a/src/main/java/cuchaz/enigma/mapping/ClassMapping.java +++ b/src/main/java/cuchaz/enigma/mapping/ClassMapping.java @@ -12,13 +12,11 @@ package cuchaz.enigma.mapping; import com.google.common.collect.Maps; -import java.io.Serializable; -import java.util.ArrayList; import java.util.Map; -public class ClassMapping implements Serializable, Comparable { +import cuchaz.enigma.throwables.MappingConflict; - private static final long serialVersionUID = -5148491146902340107L; +public class ClassMapping implements Comparable { private String m_obfFullName; private String m_obfSimpleName; @@ -70,13 +68,17 @@ public class ClassMapping implements Serializable, Comparable { return m_innerClassesByObfSimple.values(); } - public void addInnerClassMapping(ClassMapping classMapping) { - boolean obfWasAdded = m_innerClassesByObfSimple.put(classMapping.getObfSimpleName(), classMapping) == null; - assert (obfWasAdded); + public void addInnerClassMapping(ClassMapping classMapping) throws MappingConflict { + if (this.m_innerClassesByObfSimple.containsKey(classMapping.getObfSimpleName())) { + throw new MappingConflict("classes", classMapping.getObfSimpleName(), this.m_innerClassesByObfSimple.get(classMapping.getObfSimpleName()).getObfSimpleName()); + } + m_innerClassesByObfSimple.put(classMapping.getObfSimpleName(), classMapping); + if (classMapping.getDeobfName() != null) { - assert (isSimpleClassName(classMapping.getDeobfName())); - boolean deobfWasAdded = m_innerClassesByDeobf.put(classMapping.getDeobfName(), classMapping) == null; - assert (deobfWasAdded); + if (this.m_innerClassesByDeobf.containsKey(classMapping.getDeobfName())) { + throw new MappingConflict("classes", classMapping.getDeobfName(), this.m_innerClassesByDeobf.get(classMapping.getDeobfName()).getDeobfName()); + } + m_innerClassesByDeobf.put(classMapping.getDeobfName(), classMapping); } } @@ -225,17 +227,6 @@ public class ClassMapping implements Serializable, Comparable { } } - public void setFieldObfNameAndType(String oldObfName, Type obfType, String newObfName, Type newObfType) { - assert (newObfName != null); - FieldMapping fieldMapping = m_fieldsByObf.remove(getFieldKey(oldObfName, obfType)); - assert (fieldMapping != null); - fieldMapping.setObfName(newObfName); - fieldMapping.setObfType(newObfType); - boolean obfWasAdded = m_fieldsByObf.put(getFieldKey(newObfName, newObfType), fieldMapping) == null; - assert (obfWasAdded); - } - - //// METHODS //////// public Iterable methods() { @@ -307,16 +298,6 @@ public class ClassMapping implements Serializable, Comparable { } } - public void setMethodObfNameAndSignature(String oldObfName, Signature obfSignature, String newObfName, Signature newObfSignature) { - assert (newObfName != null); - MethodMapping methodMapping = m_methodsByObf.remove(getMethodKey(oldObfName, obfSignature)); - assert (methodMapping != null); - methodMapping.setObfName(newObfName); - methodMapping.setObfSignature(newObfSignature); - boolean obfWasAdded = m_methodsByObf.put(getMethodKey(newObfName, newObfSignature), methodMapping) == null; - assert (obfWasAdded); - } - //// ARGUMENTS //////// public void setArgumentName(String obfMethodName, Signature obfMethodSignature, int argumentIndex, String argumentName) { @@ -388,7 +369,4 @@ public class ClassMapping implements Serializable, Comparable { return name.indexOf('/') < 0 && name.indexOf('$') < 0; } - public ClassEntry getObfEntry() { - return new ClassEntry(m_obfFullName); - } } -- cgit v1.2.3