Skip to content
Sign in
Success

Changes

Summary

  1. Test for and correct invalid assertion in Inject.method docs (commit: 877885d) (details)
  2. Implement ModifyConstant (commit: 5186253) (details)
Commit 877885dae79241fc3a3f110dfaa2355b65c4e4e4 by Geolykt
Test 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
 (commit: 877885d)
The file was modifiedmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/mixin/MultiInjectTestMixins.java (diff)
The file was modifiedmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/targets/MultiInjectTest.java (diff)
The file was modifiedmicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/Inject.java (diff)
The file was modifiedmicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/Redirect.java (diff)
The file was modifiedmicromixin-annotations/src/main/java/org/spongepowered/asm/mixin/injection/ModifyArg.java (diff)
The file was modifiedmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/TestHarness.java (diff)
The file was modifiedmicromixin-annotations/src/main/java/com/llamalad7/mixinextras/injector/ModifyReturnValue.java (diff)
Commit 518625351877a476319df67fcd18da73cd893f8f by Geolykt
Implement 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.
 (commit: 5186253)
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/MixinMethodStub.java (diff)
The file was modifiedmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/codegen/CodegenCommon.java (diff)
The file was addedmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/targets/ModifyConstantAuxiliaryTest.java
The file was modifiedmicromixin-test-j8/src/main/resources/j8mixinconfig.json (diff)
The file was addedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/LongConstantSelector.java
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/MixinDescAnnotation.java (diff)
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/inject/TailInjectionPointSelector.java (diff)
The file was addedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/MixinModifyConstantAnnotation.java
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/util/ASMUtil.java (diff)
The file was addedmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/codegen/ModifyConstantGen.java
The file was addedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/DoubleConstantSelector.java
The file was addedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/ClassConstantSelector.java
The file was addedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/NullConstantSelector.java
The file was addedmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/mixin/ModifyConstantAuxiliaryTestMixins.java
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/api/SlicedInjectionPointSelector.java (diff)
The file was addedmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/targets/ModifyConstantTest.java
The file was addedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/FloatConstantSelector.java
The file was addedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/StringConstantSelector.java
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/MixinModifyArgAnnotation.java (diff)
The file was addedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/IntConstantSelector.java
The file was addedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/constant/WildcardConstantSelector.java
The file was modifiedmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/TestHarness.java (diff)
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/ConstantSelector.java (diff)
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/annotation/MixinAtAnnotation.java (diff)
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/inject/InvokeInjectionPointSelector.java (diff)
The file was modifiedREADME.md (diff)
The file was addedmicromixin-test-j8/src/main/java/org/stianloader/micromixin/test/j8/mixin/ModifyConstantMixins.java
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/inject/ReturnInjectionPointSelector.java (diff)
The file was modifiedmicromixin-transformer/src/main/java/org/stianloader/micromixin/internal/selectors/inject/ConstantInjectionPointSelector.java (diff)