R3944Realms/ModernFix
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fall back to interfaces when resolving capability fields

Browse Source
This commit is contained in:
Evoloxi 2026-04-13 14:47:57 +02:00
parent c64ca2e54b
commit bb4ee3b40c
1 changed files with 25 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
src/main/java/org/embeddedt/modernfix/forge/capability/CapabilityProviderDispatcherGenerator.java
View File

@ -7,6 +7,7 @@ import org.embeddedt.modernfix.ModernFix;
import org.embeddedt.modernfix.forge.capability.analysis.CapabilityAnalysisResult; import org.embeddedt.modernfix.forge.capability.analysis.CapabilityAnalysisResult;
import org.embeddedt.modernfix.forge.capability.analysis.CapabilityAnalyzer; import org.embeddedt.modernfix.forge.capability.analysis.CapabilityAnalyzer;
import org.embeddedt.modernfix.forge.capability.analysis.CapabilityRef; import org.embeddedt.modernfix.forge.capability.analysis.CapabilityRef;
import org.jetbrains.annotations.NotNull;
import org.objectweb.asm.*; import org.objectweb.asm.*;
import org.objectweb.asm.commons.GeneratorAdapter; import org.objectweb.asm.commons.GeneratorAdapter;
import org.objectweb.asm.commons.Method; import org.objectweb.asm.commons.Method;
@ -190,6 +191,8 @@ public class CapabilityProviderDispatcherGenerator {
* Resolves the actual {@link Capability} instances for all refs at class-generation time. * Resolves the actual {@link Capability} instances for all refs at class-generation time.
* Uses reflection (with {@code setAccessible}) so private fields are handled without any * Uses reflection (with {@code setAccessible}) so private fields are handled without any
* reflection bytecode appearing in the generated class. * reflection bytecode appearing in the generated class.
* <p>
* Field lookup is delegated to {@link #getRefField(Class, CapabilityRef)}
*/ */
private static List<Capability<?>> resolveCapabilityValues(LinkedHashMap<CapabilityRef, Integer> capRefIndices) { private static List<Capability<?>> resolveCapabilityValues(LinkedHashMap<CapabilityRef, Integer> capRefIndices) {
@SuppressWarnings("unchecked") @SuppressWarnings("unchecked")
@ -199,8 +202,7 @@ public class CapabilityProviderDispatcherGenerator {
try { try {
Class<?> clazz = Class.forName(ref.owner().replace('/', '.'), false, Class<?> clazz = Class.forName(ref.owner().replace('/', '.'), false,
CapabilityProviderDispatcherGenerator.class.getClassLoader()); CapabilityProviderDispatcherGenerator.class.getClassLoader());
Field field = clazz.getDeclaredField(ref.fieldName()); Field field = getRefField(clazz, ref);
field.setAccessible(true);
caps[entry.getValue()] = (Capability<?>) field.get(null); caps[entry.getValue()] = (Capability<?>) field.get(null);
} catch (ReflectiveOperationException e) { } catch (ReflectiveOperationException e) {
throw new RuntimeException("Failed to resolve capability field " + ref, e); throw new RuntimeException("Failed to resolve capability field " + ref, e);
@ -209,6 +211,27 @@ public class CapabilityProviderDispatcherGenerator {
return Arrays.asList(caps); return Arrays.asList(caps);
} }
/**
* Resolves the {@link Field} for the given {@link CapabilityRef},
* falls back to the implemented interfaces if no match is found.
*/
private static @NotNull Field getRefField(Class<?> clazz, CapabilityRef ref) throws NoSuchFieldException {
Field field = null;
try {
field = clazz.getDeclaredField(ref.fieldName());
} catch (NoSuchFieldException ignored) {
for (Class<?> iface : clazz.getInterfaces()) {
try {
field = iface.getDeclaredField(ref.fieldName());
break;
} catch (NoSuchFieldException ignored1) {}
}
}
if (field == null) throw new NoSuchFieldException(ref.fieldName());
field.setAccessible(true);
return field;
}
/** /**
* Build the dispatch list describing how each provider should be handled. * Build the dispatch list describing how each provider should be handled.
*/ */