Skip to content

Commit 55a717b

Browse files
committed
Fix patches which add files
1 parent d6512db commit 55a717b

4 files changed

Lines changed: 10 additions & 14 deletions

File tree

gradle.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,5 @@ fernflower_version=3488141
33
rdi_version=c4d6594
44
jansi_version=2.4.0
55
json_version=20220924
6-
diffpatch_version=c639936cdbed7e28c7833316161dcb7fae80d0b6
6+
diffpatch_version=cde1224
77
mapping_io_version=0.3.0

src/main/java/org/mcphackers/mcp/tasks/TaskApplyPatch.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ public static void patch(Task task, Path base, Path out, Path patches) throws IO
3434
.patchesPath(patches)
3535
.outputPath(out)
3636
.mode(PatchMode.OFFSET)
37-
.filter(p -> p.endsWith(".java"))
3837
.build();
3938
boolean success = patchOperation.doPatch();
4039
patchOperation.getSummary().print(new PrintStream(logger), false);

src/main/java/org/mcphackers/mcp/tasks/TaskDecompile.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -110,10 +110,8 @@ protected Stage[] setStages() {
110110
Files.delete(p);
111111
}
112112
}
113-
// Create Minecraft dir
114-
Files.createDirectories(ffOut.getParent());
115-
FileUtil.compress(ffOut, MCPPaths.get(mcp, SOURCE_JAR, side));
116113
Files.createDirectories(srcPath);
114+
FileUtil.compress(ffOut, MCPPaths.get(mcp, SOURCE_JAR, side));
117115
if(mcp.getOptions().getBooleanParameter(TaskParameter.OUTPUT_SRC)) {
118116
FileUtil.deletePackages(ffOut, mcp.getOptions().getStringArrayParameter(TaskParameter.IGNORED_PACKAGES));
119117
FileUtil.copyDirectory(ffOut, srcPath);

src/main/java/org/mcphackers/mcp/tasks/TaskReobfuscate.java

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -90,10 +90,11 @@ private void reobfuscate() throws IOException {
9090
if(entry.isDirectory()) {
9191
return false;
9292
}
93-
String className = entry.getName().replace(".class", "");
93+
String obfClassName = entry.getName().replace(".class", "");
9494
// Force inner classes to compare outer class hash
95-
int index;
96-
if((index = className.indexOf('$')) != -1) {
95+
String className = obfClassName;
96+
int index = className.indexOf('$');
97+
if(index != -1) {
9798
className = className.substring(0, index);
9899
}
99100
String deobfName = reversedNames.get(className);
@@ -102,10 +103,11 @@ private void reobfuscate() throws IOException {
102103
}
103104
String hash = originalHashes.get(deobfName);
104105
String hashModified = recompHashes.get(deobfName);
105-
if(hash == null) {
106-
return true;
106+
boolean extract = (hash == null) || !hash.equals(hashModified);
107+
if(extract) {
108+
System.out.println(reversedNames.get(obfClassName) + " : " + obfClassName);
107109
}
108-
else return !hash.equals(hashModified);
110+
return extract;
109111
});
110112
}
111113
}
@@ -158,9 +160,6 @@ private void modifyClassMappings(Mappings mappings, List<String> classNames, boo
158160

159161
private static Map<String, String> getPackageMappings(Map<String, String> classMappings) {
160162
Map<String, String> packageMappings = new HashMap<>();
161-
for(Entry<String, String> entry : classMappings.entrySet()) {
162-
System.out.println(entry.getKey() + " : " + entry.getValue());
163-
}
164163
for(Entry<String, String> entry : classMappings.entrySet()) {
165164
int i1 = entry.getKey().indexOf('/');
166165
int i2 = entry.getValue().indexOf('/');

0 commit comments

Comments
 (0)