Started by GitHub push by Geolykt14518625351877a476319df67fcd18da73cd893f8f518625351877a476319df67fcd18da73cd893f8frefs/remotes/origin/main518625351877a476319df67fcd18da73cd893f8f518625351877a476319df67fcd18da73cd893f8frefs/remotes/origin/main518625351877a476319df67fcd18da73cd893f8f518625351877a476319df67fcd18da73cd893f8frefs/remotes/origin/mainhttps://github.com/Starloader-project/Micromixin.githttps://stianloader.org/jenkins/job/micromixin/14/artifacthttps://stianloader.org/jenkins/job/micromixin/changeshttps://stianloader.org/jenkins/job/micromixin/14/https://stianloader.org/jenkins/job/micromixin/14/testReportfalse#143605742691micromixin #1414falsefalse1479SUCCESS1708257410824https://stianloader.org/jenkins/job/micromixin/14/micromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/Inject.javamicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/Redirect.javamicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/ModifyReturnValue.javamicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/ModifyArg.javamicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/mixin/MultiInjectTestMixins.javamicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/TestHarness.javamicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/targets/MultiInjectTest.java877885dae79241fc3a3f110dfaa2355b65c4e4e41706741638000https://stianloader.org/jenkins/user/geolyktGeolyktadmin@geolykt.deTest for and correct invalid assertion in Inject.method docs I am not exactly sure where I got the idea from that only one method is picked, however at first glance the assertion is completely incorrect and as such has been corrected to the actual (observed) behaviour. This change has been prompted by inspection of https://github.com/ModificationStation/StationAPI/blob/efa7519b1674e325c7c6c38257c3aa5b8e44493e/station-audio-loader-v0/src/main/java/net/modificationstation/stationapi/mixin/audio/BlockSoundGroupMixin.java#L21-L22 2024-01-31 23:53:58 +0100877885dae79241fc3a3f110dfaa2355b65c4e4e4Test for and correct invalid assertion in Inject.method docseditmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/mixin/MultiInjectTestMixins.javaeditmicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/ModifyArg.javaeditmicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/Redirect.javaeditmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/TestHarness.javaeditmicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/Inject.javaeditmicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/ModifyReturnValue.javaeditmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/targets/MultiInjectTest.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/ClassConstantSelector.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/inject/ReturnInjectionPointSelector.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/util/ASMUtil.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/WildcardConstantSelector.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/ConstantSelector.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/MixinModifyArgAnnotation.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/StringConstantSelector.javamicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/codegen/ModifyConstantGen.javamicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/codegen/CodegenCommon.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/FloatConstantSelector.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/NullConstantSelector.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/inject/ConstantInjectionPointSelector.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/MixinDescAnnotation.javamicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/mixin/ModifyConstantMixins.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/LongConstantSelector.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/api/SlicedInjectionPointSelector.javamicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/TestHarness.javamicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/targets/ModifyConstantAuxiliaryTest.javaREADME.mdmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/mixin/ModifyConstantAuxiliaryTestMixins.javamicromixin-test-j8/src/main/resources/j8mixinconfig.jsonmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/MixinModifyConstantAnnotation.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/DoubleConstantSelector.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/inject/TailInjectionPointSelector.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/IntConstantSelector.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/inject/InvokeInjectionPointSelector.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/MixinMethodStub.javamicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/targets/ModifyConstantTest.javamicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/MixinAtAnnotation.java518625351877a476319df67fcd18da73cd893f8f1708257231000https://stianloader.org/jenkins/user/geolyktGeolyktadmin@geolykt.deImplement ModifyConstant Also fix slices not including the last instruction within the slice. The way it is implemented is a hacky workaround, but likely well within the realm of "works". Also implements a bunch of constant selectors and moved the constant selector implementations to their own package. 2024-02-18 12:53:51 +0100518625351877a476319df67fcd18da73cd893f8fImplement ModifyConstantaddmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/ClassConstantSelector.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/MixinDescAnnotation.javaaddmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/mixin/ModifyConstantMixins.javaeditmicromixin-test-j8/src/main/resources/j8mixinconfig.jsonaddmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/targets/ModifyConstantTest.javaaddmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/StringConstantSelector.javaaddmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/targets/ModifyConstantAuxiliaryTest.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/util/ASMUtil.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/api/SlicedInjectionPointSelector.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/ConstantSelector.javaaddmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/IntConstantSelector.javaaddmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/DoubleConstantSelector.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/MixinAtAnnotation.javaaddmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/WildcardConstantSelector.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/inject/ReturnInjectionPointSelector.javaeditREADME.mdaddmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/LongConstantSelector.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/inject/InvokeInjectionPointSelector.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/MixinModifyArgAnnotation.javaaddmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/codegen/ModifyConstantGen.javaeditmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/TestHarness.javaaddmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/NullConstantSelector.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/inject/TailInjectionPointSelector.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/inject/ConstantInjectionPointSelector.javaaddmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/mixin/ModifyConstantAuxiliaryTestMixins.javaeditmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/codegen/CodegenCommon.javaaddmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/MixinModifyConstantAnnotation.javaeditmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/MixinMethodStub.javaaddmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/FloatConstantSelector.javagithttps://stianloader.org/jenkins/user/geolyktGeolyktgeolykt