BlocksMod/SpecialSource
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Optimize RemappedRuntimeInheritanceProvider to not traverse inheritance if not remapped

Browse Source 
Used to lookup the inheritance if every single class - now will return
null and allow other inheritance providers to answer instead.

Consequentially mapTypeName() has been enhanced to accept a 'default' type name:
pass the same typeName to 'pass through' unmapped names, or null if you want to
distinguish between unmapped and mapped names, as RemappedRuntimeInheritanceProvider does.
This commit is contained in:
Agaricus 2013-01-26 01:30:29 -08:00
parent 718ffc8bb5
commit 4499220c61
3 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/main/java/net/md_5/specialsource/JarRemapper.java
View File

@ -57,20 +57,19 @@ public class JarRemapper extends Remapper {
@Override
public String map(String typeName) {
return mapTypeName(typeName, jarMapping.packages, jarMapping.classes);
return mapTypeName(typeName, jarMapping.packages, jarMapping.classes, typeName);
}
public static String mapTypeName(String typeName, Map<String, String> packageMap, Map<String, String> classMap) {
public static String mapTypeName(String typeName, Map<String, String> packageMap, Map<String, String> classMap, String defaultIfUnmapped) {
int index = typeName.indexOf('$');
String key = (index == -1) ? typeName : typeName.substring(0, index);
String mapped = mapClassName(key, packageMap, classMap);
return mapped != null ? mapped + (index == -1 ? "" : typeName.substring(index, typeName.length())) : typeName;
return mapped != null ? mapped + (index == -1 ? "" : typeName.substring(index, typeName.length())) : defaultIfUnmapped;
}
/**
* Helper method to map a class name by package (prefix) or class (exact)
* map
*/
private static String mapClassName(String className, Map<String, String> packageMap, Map<String, String> classMap) {
if (packageMap != null) {

2
src/main/java/net/md_5/specialsource/MethodDescriptorTransformer.java
View File

@ -57,7 +57,7 @@ public class MethodDescriptorTransformer {
String className = rest.substring(1, end);
i += className.length() + 1;
String newClassName = JarRemapper.mapTypeName(className, packageMap, classMap);
String newClassName = JarRemapper.mapTypeName(className, packageMap, classMap, className);
output.append("L").append(newClassName).append(";");
break;

11
src/main/java/net/md_5/specialsource/RemappedRuntimeInheritanceProvider.java
View File

@ -31,7 +31,14 @@ public class RemappedRuntimeInheritanceProvider extends RuntimeInheritanceProvid
@Override
public List<String> getParents(String before) {
// Remap the input (example: cb -> obf)
String after = JarRemapper.mapTypeName(before, jarMapping.packages, jarMapping.classes);
// If the type is not mapped, return immediately
String after = JarRemapper.mapTypeName(before, jarMapping.packages, jarMapping.classes, null);
if (after == null) {
if (verbose) {
System.out.println("RemappedRuntimeInheritanceProvider doesn't know about "+before);
}
return null;
}
if (verbose) {
System.out.println("RemappedRuntimeInheritanceProvider getParents "+before+" -> "+after);
@ -48,7 +55,7 @@ public class RemappedRuntimeInheritanceProvider extends RuntimeInheritanceProvid
// Un-remap the output (example: obf -> cb)
List<String> afterParents = new ArrayList<String>();
for (String beforeParent : beforeParents) {
String afterParent = JarRemapper.mapTypeName(beforeParent, inverseJarMapping.packages, inverseJarMapping.classes);
String afterParent = JarRemapper.mapTypeName(beforeParent, inverseJarMapping.packages, inverseJarMapping.classes, beforeParent);
if (verbose) {
System.out.println("- " + beforeParent + " -> " + afterParent);
}