mirrors/Flywheel
0
0
Fork 0
mirror of https://github.com/Jozufozu/Flywheel.git synced 2024-11-14 14:33:57 +01:00
Code Issues Packages Projects Releases Wiki Activity

DidObfuscate

Browse Source 
- Fix RenderSystemMixin on fabric
- setShaderFogShape's arguments need to be remapped, but the name of the
  function should not be. Fortunately mixin allows matching by function
  name alone
This commit is contained in:
Jozufozu 2024-10-28 13:36:39 -07:00
parent 03f94fe88c
commit 84e27867d3
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
common/src/backend/java/dev/engine_room/flywheel/backend/mixin/RenderSystemMixin.java
View File

@ -27,7 +27,10 @@ abstract class RenderSystemMixin {
FogUniforms.update(); FogUniforms.update();
} }
@Inject(method = "setShaderFogShape(Lcom/mojang/blaze3d/shaders/FogShape;)V", at = @At("RETURN")) // Fabric fails to resolve the mixin in prod when the full signature is specified.
// I suspect it's because this method references a class name in its signature,
// and that needs to be remapped while the function names in RenderSystem are marked with @DontObfuscate.
@Inject(method = "setShaderFogShape", at = @At("RETURN"))
private static void flywheel$onSetFogShape(CallbackInfo ci) { private static void flywheel$onSetFogShape(CallbackInfo ci) {
FogUniforms.update(); FogUniforms.update();
} }