Started by GitHub push by Geolykt102dbc31fd12fb4bbe972e7550b7b16d24e9e79bf85dbc31fd12fb4bbe972e7550b7b16d24e9e79bf85refs/remotes/origin/maindbc31fd12fb4bbe972e7550b7b16d24e9e79bf85dbc31fd12fb4bbe972e7550b7b16d24e9e79bf85refs/remotes/origin/maindbc31fd12fb4bbe972e7550b7b16d24e9e79bf85dbc31fd12fb4bbe972e7550b7b16d24e9e79bf85refs/remotes/origin/mainhttps://github.com/stianloader/Micromixin.gitfalse#1028078342691micromixin #102102falsefalse102401SUCCESS1775266468721https://stianloader.org/jenkins/view/all/job/micromixin/102/micromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/targets/InjectMixinsTarget.javamicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/Inject.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/VirtualConstructorMergeAnnotation.java.gitignoremicromixin-testing-suite-neo/build.gradlemicromixin-transformer/build.gradlemicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/mixins/LocalCaptureMixins.javamicromixin-backports/build.gradlemicromixin-test-j8/pom.xmlgradle/wrapper/gradle-wrapper.propertiesmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/targets/LocalCaptureMixinsTarget.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/MixinStub.javamicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/MicromixinTestNeo.javamicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/mixins/InjectMixins.javamicromixin-annotations/build.gradlemicromixin-testing-suite-neo/src/main/java/module-info.javamicromixin-transformer/src/test/java/org/stianloader/micromixin/test/RedirectTest.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinInjectAnnotation.javamicromixin-testing-suite-neo/micromixin-test-config-gradle.jsonmicromixin-testing-suite-neo/src/main/resources/micromixin-test-neo-mixins.jsondbc31fd12fb4bbe972e7550b7b16d24e9e79bf851775266447000https://stianloader.org/jenkins/user/geolyktGeolyktadmin@geolykt.deCorrectly handle local capture It's a bit strange because some of this logic should have historically worked, but apparently not? I guess there might have been one or the other blunder in the old test suite for local capture - I'd have to look into it anew maybe, though I suppose it's partially down to a regression within how local variable detection is performed in conjunction to when the CallbackInfos are injected into the code (in other words, we now inject it earlier than we used to, so now can capture the CallbackInfo which made this regression apparent - though it went uncaught because the old test suite wasn't run since then). Regardless, this solves following issues with local capture: - Argument capture could be omitted when capturing locals is enabled. But only when not capturing any locals. This was incorrect behaviour. All arguments must get captured when local capture is enabled. - Local capture overflow would produce a cryptic ArrayIndexOutOfBoundsException This has been replaced with a more descriptive exception message. - Local capture underflow wasn't supported, even though it was specified in the specs. This has been amended: Trailing locals can now be omitted. - Not capturing arguments when capturing locals would cause seemingly unrelated errors. This has been amended: Proper hints are now added to the exception message detailing possible causes. - The inability to capture local variables without capturing arguments was not documented. This has also been amended: The javadocs now properly state that argument capture is necessary for local capture. Also solves a few issues within the test suite regarding failing tests. 2026-04-04 03:34:07 +0200dbc31fd12fb4bbe972e7550b7b16d24e9e79bf85Correctly handle local captureedit.gitignoreeditmicromixin-testing-suite-neo/build.gradleaddmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/mixins/LocalCaptureMixins.javaeditmicromixin-testing-suite-neo/src/main/resources/micromixin-test-neo-mixins.jsoneditmicromixin-transformer/src/test/java/org/stianloader/micromixin/test/RedirectTest.javaeditmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/MicromixinTestNeo.javaeditgradle/wrapper/gradle-wrapper.propertieseditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/VirtualConstructorMergeAnnotation.javaeditmicromixin-transformer/build.gradleeditmicromixin-test-j8/pom.xmleditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/MixinStub.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/transform/internal/annotation/MixinInjectAnnotation.javaaddmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/targets/LocalCaptureMixinsTarget.javaeditmicromixin-annotations/build.gradleeditmicromixin-testing-suite-neo/src/main/java/module-info.javaeditmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/targets/InjectMixinsTarget.javaeditmicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/Inject.javaeditmicromixin-testing-suite-neo/micromixin-test-config-gradle.jsoneditmicromixin-testing-suite-neo/src/main/java/org/stianloader/micromixin/testneo/testenv/mixins/InjectMixins.javaeditmicromixin-backports/build.gradlegithttps://stianloader.org/jenkins/user/geolyktGeolykt