Skip to content
Success

Changes

Summary

  1. Fix Redirect not properly pushing this down the operand stack (commit: b543e4c) (details)
Commit b543e4c305be3ffebe884d0500afd814e9ed6152 by Geolykt
Fix Redirect not properly pushing this down the operand stack

It would occasionally cause category 2 computational types to be cut off, which
is behaviour not tolerated by the Verifier. Now, Redirect uses almost the same
logic as ModifyArg, although ModifyArg's stack head moving logic had to be slightly
adapted in order to accomodate scenarios where nothing needs to be pushed down the
stack. This is primiarily an issue as for Redirect you can be quite sure there is
background (unless you'd want to compute it but that might be a bit on the
expensive side of things for little benefit)
(commit: b543e4c)
The file was modifiedmicromixin-transformer/src/test/java/org/stianloader/micromixin/test/ASMUtilTest.java (diff)
The file was modifiedmicromixin-transformer/build.gradle (diff)
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinRedirectAnnotation.java (diff)
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/util/ASMUtil.java (diff)