From 935f275ecf82c7b52713559f988f075f8cbec403 Mon Sep 17 00:00:00 2001 From: joe Date: Fri, 2 Jan 2026 20:29:35 +0000 Subject: [PATCH 1/6] Use local variable names for all mixin locals --- .../fabric/mixin/block/LivingEntityMixin.java | 2 +- .../client/RenderRegionCacheMixin.java | 2 +- .../gametest/input/GlCommandEncoderMixin.java | 2 +- .../gametest/world/CreateWorldScreenMixin.java | 2 +- .../mixin/content/registry/AxeItemMixin.java | 2 +- .../content/registry/FuelValuesMixin.java | 2 +- .../registry/WalkNodeEvaluatorMixin.java | 2 +- .../client/ClientPacketListenerMixin.java | 2 +- .../datagen/client/ModelProviderMixin.java | 4 ++-- .../mixin/datagen/TagsProviderMixin.java | 4 ++-- .../AllCraftingRecipeJsonBuildersMixin.java | 2 +- .../SmithingTransformRecipeBuilderMixin.java | 2 +- .../recipe/SmithingTrimRecipeBuilderMixin.java | 2 +- .../recipe/SpecialRecipeBuilderMixin.java | 2 +- .../mixin/entity/event/LivingEntityMixin.java | 2 +- .../fabric/mixin/entity/event/MobMixin.java | 2 +- .../interaction/client/MinecraftMixin.java | 2 +- .../client/MultiPlayerGameModeMixin.java | 2 +- .../ServerGamePacketListenerImplMixin.java | 4 ++-- .../interaction/ServerPlayerGameModeMixin.java | 4 ++-- .../StructureTemplateManagerMixin.java | 2 +- .../fabric/mixin/item/AnvilMenuMixin.java | 2 +- .../fabric/mixin/item/ItemStackMixin.java | 18 +++++++++--------- .../lifecycle/client/LevelChunkMixin.java | 4 ++-- .../mixin/event/lifecycle/ChunkMapMixin.java | 2 +- .../event/lifecycle/LivingEntityMixin.java | 2 +- .../event/lifecycle/MinecraftServerMixin.java | 2 +- .../lifecycle/server/LevelChunkMixin.java | 4 ++-- .../loot/ReloadableServerRegistriesMixin.java | 4 ++-- .../fabric/mixin/menu/ServerPlayerMixin.java | 2 +- .../client/message/ChatListenerMixin.java | 2 +- .../message/ClientPacketListenerMixin.java | 4 ++-- .../model/loading/ModelManagerMixin.java | 6 +++--- .../mixin/networking/BundlePacketMixin.java | 2 +- .../mixin/networking/ConnectionMixin.java | 4 ++-- .../mixin/networking/IdDispatchCodecMixin.java | 2 +- .../object/builder/EntityTypeBuilderMixin.java | 2 +- .../client/particle/TerrainParticleMixin.java | 5 +++-- .../fabric/mixin/particle/EntityMixin.java | 2 +- .../mixin/recipe/sync/RecipeMapMixin.java | 2 +- .../mixin/registry/sync/RegistriesMixin.java | 4 ++-- .../block/model/SimpleModelWrapperMixin.java | 2 +- .../particle/ScreenEffectRendererMixin.java | 4 ++-- .../block/render/ItemFrameRendererMixin.java | 2 +- .../renderer/item/BlockModelWrapperMixin.java | 2 +- .../renderer/ItemStackRenderStateMixin.java | 4 ++-- .../indigo/renderer/SectionCompilerMixin.java | 8 ++++---- .../fluid/LiquidBlockRendererMixin.java | 10 +++++----- .../mixin/client/rendering/CapeLayerMixin.java | 2 +- .../DebugOptionsScreenOptionListMixin.java | 2 +- .../rendering/EntityRenderDispatcherMixin.java | 2 +- .../client/rendering/GuiRendererMixin.java | 2 +- .../rendering/LayerDefinitionsMixin.java | 2 +- .../rendering/LivingEntityRendererMixin.java | 2 +- .../mixin/resource/conditions/PackMixin.java | 4 ++-- ...gistryLoadTaskPendingRegistrationMixin.java | 2 +- .../SimpleJsonResourceReloadListenerMixin.java | 4 ++-- .../client/CreateWorldScreenMixin.java | 3 ++- .../mixin/resource/DataPackCommandMixin.java | 2 +- .../mixin/resource/PackRepositoryMixin.java | 6 +++--- .../ReloadableServerResourcesMixin.java | 6 +++--- .../resource/SimpleReloadInstanceMixin.java | 7 ++++--- .../fabric/mixin/tag/TagLoaderMixin.java | 2 +- .../fabric/mixin/transfer/BucketItemMixin.java | 2 +- .../mixin/transfer/CrafterBlockMixin.java | 2 +- .../mixin/transfer/HopperBlockEntityMixin.java | 4 ++-- 66 files changed, 108 insertions(+), 105 deletions(-) diff --git a/fabric-block-api-v1/src/main/java/net/fabricmc/fabric/mixin/block/LivingEntityMixin.java b/fabric-block-api-v1/src/main/java/net/fabricmc/fabric/mixin/block/LivingEntityMixin.java index 542a6f2f499..eba98e9bc38 100644 --- a/fabric-block-api-v1/src/main/java/net/fabricmc/fabric/mixin/block/LivingEntityMixin.java +++ b/fabric-block-api-v1/src/main/java/net/fabricmc/fabric/mixin/block/LivingEntityMixin.java @@ -38,7 +38,7 @@ abstract class LivingEntityMixin { allow = 1, cancellable = true ) - private void allowTaggedBlocksForTrapdoorClimbing(BlockPos pos, BlockState state, CallbackInfoReturnable info, @Local(ordinal = 1) BlockState belowState) { + private void allowTaggedBlocksForTrapdoorClimbing(BlockPos pos, BlockState state, CallbackInfoReturnable info, @Local(name = "belowState") BlockState belowState) { if (belowState.is(BlockFunctionalityTags.CAN_CLIMB_TRAPDOOR_ABOVE)) { if (belowState.getBlock() instanceof LadderBlock) { // Check that the ladder and trapdoor are placed in the same direction. diff --git a/fabric-block-getter-api-v2/src/client/java/net/fabricmc/fabric/mixin/blockgetter/client/RenderRegionCacheMixin.java b/fabric-block-getter-api-v2/src/client/java/net/fabricmc/fabric/mixin/blockgetter/client/RenderRegionCacheMixin.java index 0a34c075723..eb5697ad235 100644 --- a/fabric-block-getter-api-v2/src/client/java/net/fabricmc/fabric/mixin/blockgetter/client/RenderRegionCacheMixin.java +++ b/fabric-block-getter-api-v2/src/client/java/net/fabricmc/fabric/mixin/blockgetter/client/RenderRegionCacheMixin.java @@ -50,7 +50,7 @@ public abstract class RenderRegionCacheMixin { private static final Logger LOGGER = LoggerFactory.getLogger(RenderRegionCacheMixin.class); @Inject(method = "createRegion", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/renderer/chunk/RenderRegionCache;getSectionDataCopy(Lnet/minecraft/world/level/Level;III)Lnet/minecraft/client/renderer/chunk/SectionCopy;")) - private void copyDataForChunk(Level level, long packedChunkPos, CallbackInfoReturnable cir, @Share("dataMap") LocalRef> mapRef, @Local(ordinal = 11) int x, @Local(ordinal = 10) int y, @Local(ordinal = 9) int z) { + private void copyDataForChunk(Level level, long packedChunkPos, CallbackInfoReturnable cir, @Share("dataMap") LocalRef> mapRef, @Local(name = "regionSectionX") int x, @Local(name = "regionSectionY") int y, @Local(name = "regionSectionZ") int z) { // Hash maps in chunks should generally not be modified outside of client thread // but does happen in practice, due to mods or inconsistent vanilla behaviors, causing // CMEs when we iterate the map. (Vanilla does not iterate these maps when it builds diff --git a/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/input/GlCommandEncoderMixin.java b/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/input/GlCommandEncoderMixin.java index 5ba1cb78874..7e04f52f28e 100644 --- a/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/input/GlCommandEncoderMixin.java +++ b/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/input/GlCommandEncoderMixin.java @@ -32,7 +32,7 @@ @Mixin(GlCommandEncoder.class) public class GlCommandEncoderMixin { @WrapOperation(method = "presentTexture", at = @At(value = "INVOKE", target = "Lcom/mojang/blaze3d/opengl/DirectStateAccess;blitFrameBuffers(IIIIIIIIIIII)V")) - private void blitFrameBuffer(DirectStateAccess manager, int readFramebuffer, int drawFramebuffer, int srcX0, int srcY0, int srcX1, int srcY1, int dstX0, int dstY0, int dstX1, int dstY1, int mask, int filter, Operation original, @Local(argsOnly = true) GpuTextureView gpuTextureView) { + private void blitFrameBuffer(DirectStateAccess manager, int readFramebuffer, int drawFramebuffer, int srcX0, int srcY0, int srcX1, int srcY1, int dstX0, int dstY0, int dstX1, int dstY1, int mask, int filter, Operation original, @Local(argsOnly = true, name = "textureView") GpuTextureView gpuTextureView) { if (gpuTextureView.texture() == Minecraft.getInstance().getMainRenderTarget().getColorTexture()) { WindowHooks window = ((WindowHooks) (Object) Minecraft.getInstance().getWindow()); dstX1 = window.fabric_getRealFramebufferWidth(); diff --git a/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/world/CreateWorldScreenMixin.java b/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/world/CreateWorldScreenMixin.java index 0e1c43b5b83..0e050ee9dae 100644 --- a/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/world/CreateWorldScreenMixin.java +++ b/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/world/CreateWorldScreenMixin.java @@ -38,7 +38,7 @@ @Mixin(CreateWorldScreen.class) public class CreateWorldScreenMixin { @Inject(method = "onCreate", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/gui/screens/worldselection/WorldOpenFlows;confirmWorldCreation(Lnet/minecraft/client/Minecraft;Lnet/minecraft/client/gui/screens/worldselection/CreateWorldScreen;Lcom/mojang/serialization/Lifecycle;Ljava/lang/Runnable;Z)V"), cancellable = true) - private void createLevelDataForServers(CallbackInfo ci, @Local LayeredRegistryAccess registries, @Local PrimaryLevelData levelData) { + private void createLevelDataForServers(CallbackInfo ci, @Local(name = "finalLayers") LayeredRegistryAccess registries, @Local(name = "worldData") PrimaryLevelData levelData) { if (DedicatedServerImplUtil.saveLevelDataTo != null) { CompoundTag levelDatInner = levelData.createTag(registries.compositeAccess(), null); CompoundTag levelDat = new CompoundTag(); diff --git a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/AxeItemMixin.java b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/AxeItemMixin.java index 9d609e637de..e785e87d105 100644 --- a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/AxeItemMixin.java +++ b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/AxeItemMixin.java @@ -33,7 +33,7 @@ @Mixin(AxeItem.class) public class AxeItemMixin { @ModifyArg(method = "getStripped", at = @At(value = "INVOKE", target = "Ljava/util/Optional;map(Ljava/util/function/Function;)Ljava/util/Optional;")) - private Function handleCustomStrippingBehavior(Function mapper, @Local(argsOnly = true) BlockState state) { + private Function handleCustomStrippingBehavior(Function mapper, @Local(argsOnly = true, name = "state") BlockState state) { StrippableBlockRegistry.StrippingTransformer transformer = StrippableBlockRegistryImpl.getTransformer(state.getBlock()); if (transformer != null) { diff --git a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/FuelValuesMixin.java b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/FuelValuesMixin.java index 57da5804d33..904cf93a831 100644 --- a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/FuelValuesMixin.java +++ b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/FuelValuesMixin.java @@ -49,7 +49,7 @@ public abstract class FuelValuesMixin { ), allow = 1 ) - private static FuelValues.Builder build(FuelValues.Builder builder, TagKey tag, Operation operation, @Local(argsOnly = true) HolderLookup.Provider registries, @Local(argsOnly = true) FeatureFlagSet features, @Local(argsOnly = true) int baseSmeltTime) { + private static FuelValues.Builder build(FuelValues.Builder builder, TagKey tag, Operation operation, @Local(argsOnly = true, name = "registries") HolderLookup.Provider registries, @Local(argsOnly = true, name = "enabledFeatures") FeatureFlagSet features, @Local(argsOnly = true, name = "baseUnit") int baseSmeltTime) { final var context = new FuelRegistryEventsContextImpl(registries, features, baseSmeltTime); FuelValueEvents.BUILD.invoker().build(builder, context); diff --git a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/WalkNodeEvaluatorMixin.java b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/WalkNodeEvaluatorMixin.java index 110592cf721..d2a2ea79b55 100644 --- a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/WalkNodeEvaluatorMixin.java +++ b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/WalkNodeEvaluatorMixin.java @@ -37,7 +37,7 @@ public class WalkNodeEvaluatorMixin { * Overrides the node type for the specified position, if the position is a direct target in a path. */ @Inject(method = "getPathTypeFromState", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/state/BlockState;getBlock()Lnet/minecraft/world/level/block/Block;"), cancellable = true) - private static void getCommonNodeType(BlockGetter level, BlockPos pos, CallbackInfoReturnable cir, @Local BlockState state) { + private static void getCommonNodeType(BlockGetter level, BlockPos pos, CallbackInfoReturnable cir, @Local(name = "blockState") BlockState state) { PathType pathType = LandPathTypeRegistry.getPathType(state, level, pos, false); if (pathType != null) { diff --git a/fabric-data-attachment-api-v1/src/client/java/net/fabricmc/fabric/mixin/attachment/client/ClientPacketListenerMixin.java b/fabric-data-attachment-api-v1/src/client/java/net/fabricmc/fabric/mixin/attachment/client/ClientPacketListenerMixin.java index bc6e4333477..e3024244ac2 100644 --- a/fabric-data-attachment-api-v1/src/client/java/net/fabricmc/fabric/mixin/attachment/client/ClientPacketListenerMixin.java +++ b/fabric-data-attachment-api-v1/src/client/java/net/fabricmc/fabric/mixin/attachment/client/ClientPacketListenerMixin.java @@ -44,7 +44,7 @@ abstract class ClientPacketListenerMixin { from = @At(value = "INVOKE", target = "Lnet/minecraft/client/multiplayer/ClientPacketListener;startWaitingForNewLevel(Lnet/minecraft/client/player/LocalPlayer;Lnet/minecraft/client/multiplayer/ClientLevel;Lnet/minecraft/client/gui/screens/LevelLoadingScreen$Reason;)V") ) ) - private void copyAttachmentsOnClientRespawn(Minecraft client, LocalPlayer newPlayer, Operation init, ClientboundRespawnPacket packet, @Local(ordinal = 0) LocalPlayer oldPlayer) { + private void copyAttachmentsOnClientRespawn(Minecraft client, LocalPlayer newPlayer, Operation init, ClientboundRespawnPacket packet, @Local(name = "oldPlayer") LocalPlayer oldPlayer) { /* * The KEEP_ATTRIBUTES flag is not set on a death respawn, and set in all other cases */ diff --git a/fabric-data-generation-api-v1/src/client/java/net/fabricmc/fabric/mixin/datagen/client/ModelProviderMixin.java b/fabric-data-generation-api-v1/src/client/java/net/fabricmc/fabric/mixin/datagen/client/ModelProviderMixin.java index 8fcfaa41925..d2183c5966c 100644 --- a/fabric-data-generation-api-v1/src/client/java/net/fabricmc/fabric/mixin/datagen/client/ModelProviderMixin.java +++ b/fabric-data-generation-api-v1/src/client/java/net/fabricmc/fabric/mixin/datagen/client/ModelProviderMixin.java @@ -73,8 +73,8 @@ private void registerItemModels(ItemModelGenerators instance, Operation or @Inject(method = "run", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/data/models/BlockModelGenerators;run()V")) private void setFabricPackOutput(CachedOutput output, CallbackInfoReturnable> cir, - @Local ModelProvider.BlockStateGeneratorCollector blockStateGeneratorCollector, - @Local ModelProvider.ItemInfoCollector itemInfoCollectors) { + @Local(name = "blockStateGenerators") ModelProvider.BlockStateGeneratorCollector blockStateGeneratorCollector, + @Local(name = "itemModels") ModelProvider.ItemInfoCollector itemInfoCollectors) { ((FabricModelProviderDefinitions) blockStateGeneratorCollector).setFabricPackOutput(fabricPackOutput); ((FabricModelProviderDefinitions) itemInfoCollectors).setFabricPackOutput(fabricPackOutput); ((FabricItemAssetDefinitions) itemInfoCollectors).fabric_setProcessedBlocks(blockStateGeneratorCollector.generators.keySet()); diff --git a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/TagsProviderMixin.java b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/TagsProviderMixin.java index 7c918e92817..47e9ae0e7c8 100644 --- a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/TagsProviderMixin.java +++ b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/TagsProviderMixin.java @@ -58,7 +58,7 @@ private void initPathResolver(PackOutput output, ResourceKey(Ljava/util/List;Z)V"), index = 1) - private boolean addReplaced(boolean replaced, @Local TagBuilder tagBuilder) { + private boolean addReplaced(boolean replaced, @Local(name = "builder") TagBuilder tagBuilder) { if (tagBuilder instanceof FabricTagBuilder fabricTagBuilder) { return fabricTagBuilder.fabric_isReplaced(); } @@ -68,7 +68,7 @@ private boolean addReplaced(boolean replaced, @Local TagBuilder tagBuilder) { @SuppressWarnings("unchecked") @WrapOperation(method = "lambda$run$2", at = @At(value = "INVOKE", target = "Ljava/util/concurrent/CompletableFuture;allOf([Ljava/util/concurrent/CompletableFuture;)Ljava/util/concurrent/CompletableFuture;")) - private CompletableFuture addTagAliasGroupBuilders(CompletableFuture[] futures, Operation> original, @Local(argsOnly = true) CachedOutput writer) { + private CompletableFuture addTagAliasGroupBuilders(CompletableFuture[] futures, Operation> original, @Local(argsOnly = true, name = "cache") CachedOutput writer) { if ((Object) this instanceof FabricTagsProvider) { // Note: no pattern matching instanceof so that we can cast directly to FabricTagsProvider instead of a wildcard Map.AliasGroupBuilder> builders = ((FabricTagsProvider) (Object) this).getAliasGroupBuilders(); diff --git a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/AllCraftingRecipeJsonBuildersMixin.java b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/AllCraftingRecipeJsonBuildersMixin.java index 070402b2078..e1e03e49ba0 100644 --- a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/AllCraftingRecipeJsonBuildersMixin.java +++ b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/AllCraftingRecipeJsonBuildersMixin.java @@ -37,7 +37,7 @@ TransmuteRecipeBuilder.class, }) abstract class AllCraftingRecipeJsonBuildersMixin { - @ModifyVariable(method = "save(Lnet/minecraft/data/recipes/RecipeOutput;Lnet/minecraft/resources/ResourceKey;)V", at = @At("HEAD"), argsOnly = true) + @ModifyVariable(method = "save(Lnet/minecraft/data/recipes/RecipeOutput;Lnet/minecraft/resources/ResourceKey;)V", at = @At("HEAD"), argsOnly = true, name = "id") private ResourceKey> modifyRecipeKey(ResourceKey> recipeKey, RecipeOutput output) { return ResourceKey.create(recipeKey.registryKey(), output.getRecipeIdentifier(recipeKey.identifier())); } diff --git a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/SmithingTransformRecipeBuilderMixin.java b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/SmithingTransformRecipeBuilderMixin.java index 9762f432cc5..6b858ea8b90 100644 --- a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/SmithingTransformRecipeBuilderMixin.java +++ b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/SmithingTransformRecipeBuilderMixin.java @@ -27,7 +27,7 @@ @Mixin(SmithingTransformRecipeBuilder.class) abstract class SmithingTransformRecipeBuilderMixin { - @ModifyVariable(method = "save(Lnet/minecraft/data/recipes/RecipeOutput;Lnet/minecraft/resources/ResourceKey;)V", at = @At("HEAD"), argsOnly = true) + @ModifyVariable(method = "save(Lnet/minecraft/data/recipes/RecipeOutput;Lnet/minecraft/resources/ResourceKey;)V", at = @At("HEAD"), argsOnly = true, name = "id") private ResourceKey> modifyRecipeKey(ResourceKey> recipeKey, RecipeOutput output) { return ResourceKey.create(recipeKey.registryKey(), output.getRecipeIdentifier(recipeKey.identifier())); } diff --git a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/SmithingTrimRecipeBuilderMixin.java b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/SmithingTrimRecipeBuilderMixin.java index 5e678576926..78f66ae356a 100644 --- a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/SmithingTrimRecipeBuilderMixin.java +++ b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/SmithingTrimRecipeBuilderMixin.java @@ -27,7 +27,7 @@ @Mixin(SmithingTrimRecipeBuilder.class) abstract class SmithingTrimRecipeBuilderMixin { - @ModifyVariable(method = "save(Lnet/minecraft/data/recipes/RecipeOutput;Lnet/minecraft/resources/ResourceKey;)V", at = @At("HEAD"), argsOnly = true) + @ModifyVariable(method = "save(Lnet/minecraft/data/recipes/RecipeOutput;Lnet/minecraft/resources/ResourceKey;)V", at = @At("HEAD"), argsOnly = true, name = "id") private ResourceKey> modifyRecipeKey(ResourceKey> recipeKey, RecipeOutput output) { return ResourceKey.create(recipeKey.registryKey(), output.getRecipeIdentifier(recipeKey.identifier())); } diff --git a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/SpecialRecipeBuilderMixin.java b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/SpecialRecipeBuilderMixin.java index d7e5279db9c..c1976711aa2 100644 --- a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/SpecialRecipeBuilderMixin.java +++ b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/recipe/SpecialRecipeBuilderMixin.java @@ -27,7 +27,7 @@ @Mixin(SpecialRecipeBuilder.class) abstract class SpecialRecipeBuilderMixin { - @ModifyVariable(method = "save(Lnet/minecraft/data/recipes/RecipeOutput;Lnet/minecraft/resources/ResourceKey;)V", at = @At("HEAD"), argsOnly = true) + @ModifyVariable(method = "save(Lnet/minecraft/data/recipes/RecipeOutput;Lnet/minecraft/resources/ResourceKey;)V", at = @At("HEAD"), argsOnly = true, name = "id") private ResourceKey> modifyRecipeKey(ResourceKey> recipeKey, RecipeOutput output) { return ResourceKey.create(recipeKey.registryKey(), output.getRecipeIdentifier(recipeKey.identifier())); } diff --git a/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/LivingEntityMixin.java b/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/LivingEntityMixin.java index 61f297b18b4..242de8984b0 100644 --- a/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/LivingEntityMixin.java +++ b/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/LivingEntityMixin.java @@ -85,7 +85,7 @@ private void beforeDamage(ServerLevel level, DamageSource source, float amount, } @Inject(method = "hurtServer", at = @At("TAIL")) - private void afterDamage(ServerLevel level, DamageSource source, float amount, CallbackInfoReturnable cir, @Local(ordinal = 1) float dealt, @Local(ordinal = 0) boolean blocked) { + private void afterDamage(ServerLevel level, DamageSource source, float amount, CallbackInfoReturnable cir, @Local(name = "originalDamage") float dealt, @Local(name = "blocked") boolean blocked) { if (!isDeadOrDying()) { ServerLivingEntityEvents.AFTER_DAMAGE.invoker().afterDamage((LivingEntity) (Object) this, source, dealt, amount, blocked); } diff --git a/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/MobMixin.java b/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/MobMixin.java index 3e40abd1c43..976afe76ab2 100644 --- a/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/MobMixin.java +++ b/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/MobMixin.java @@ -30,7 +30,7 @@ @Mixin(Mob.class) public class MobMixin { @ModifyArg(method = "convertTo(Lnet/minecraft/world/entity/EntityType;Lnet/minecraft/world/entity/ConversionParams;Lnet/minecraft/world/entity/EntitySpawnReason;Lnet/minecraft/world/entity/ConversionParams$AfterConversion;)Lnet/minecraft/world/entity/Mob;", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/level/ServerLevel;addFreshEntity(Lnet/minecraft/world/entity/Entity;)Z")) - private Entity afterEntityConverted(Entity converted, @Local(argsOnly = true) ConversionParams conversionContext) { + private Entity afterEntityConverted(Entity converted, @Local(argsOnly = true, name = "params") ConversionParams conversionContext) { ServerLivingEntityEvents.MOB_CONVERSION.invoker().onConversion((Mob) (Object) this, (Mob) converted, conversionContext); return converted; } diff --git a/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MinecraftMixin.java b/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MinecraftMixin.java index 9b4addb4e7a..61ba2ca4ced 100644 --- a/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MinecraftMixin.java +++ b/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MinecraftMixin.java @@ -74,7 +74,7 @@ public abstract class MinecraftMixin { method = "startUseItem", cancellable = true ) - private void injectUseEntityCallback(CallbackInfo ci, @Local InteractionHand hand, @Local EntityHitResult hitResult, @Local Entity entity) { + private void injectUseEntityCallback(CallbackInfo ci, @Local(name = "hand") InteractionHand hand, @Local(name = "entityHit") EntityHitResult hitResult, @Local(name = "entity") Entity entity) { InteractionResult result = UseEntityCallback.EVENT.invoker().interact(player, player.level(), hand, entity, hitResult); if (result != InteractionResult.PASS) { diff --git a/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MultiPlayerGameModeMixin.java b/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MultiPlayerGameModeMixin.java index 936c890f59f..f4cc1366467 100644 --- a/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MultiPlayerGameModeMixin.java +++ b/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MultiPlayerGameModeMixin.java @@ -91,7 +91,7 @@ private void fabric_fireAttackBlockCallback(BlockPos pos, Direction direction, C } @Inject(method = "destroyBlock", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/Block;destroy(Lnet/minecraft/world/level/LevelAccessor;Lnet/minecraft/core/BlockPos;Lnet/minecraft/world/level/block/state/BlockState;)V")) - private void fabric$onBlockBroken(BlockPos pos, CallbackInfoReturnable cir, @Local BlockState blockState) { + private void fabric$onBlockBroken(BlockPos pos, CallbackInfoReturnable cir, @Local(name = "oldState") BlockState blockState) { ClientPlayerBlockBreakEvents.AFTER.invoker().afterBlockBreak(minecraft.level, minecraft.player, pos, blockState); } diff --git a/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerGamePacketListenerImplMixin.java b/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerGamePacketListenerImplMixin.java index 93ace511f66..85b7fc6baa0 100644 --- a/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerGamePacketListenerImplMixin.java +++ b/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerGamePacketListenerImplMixin.java @@ -53,7 +53,7 @@ private void tryPickItem(ItemStack stack) { } @WrapOperation(method = "handlePickItemFromBlock", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/state/BlockState;getCloneItemStack(Lnet/minecraft/world/level/LevelReader;Lnet/minecraft/core/BlockPos;Z)Lnet/minecraft/world/item/ItemStack;")) - public ItemStack onPickItemFromBlock(BlockState state, LevelReader level, BlockPos pos, boolean includeData, Operation operation, @Local(argsOnly = true) ServerboundPickItemFromBlockPacket packet) { + public ItemStack onPickItemFromBlock(BlockState state, LevelReader level, BlockPos pos, boolean includeData, Operation operation, @Local(argsOnly = true, name = "packet") ServerboundPickItemFromBlockPacket packet) { ItemStack stack = PlayerPickItemEvents.BLOCK.invoker().onPickItemFromBlock(player, pos, state, packet.includeData()); if (stack == null) { @@ -67,7 +67,7 @@ public ItemStack onPickItemFromBlock(BlockState state, LevelReader level, BlockP } @WrapOperation(method = "handlePickItemFromEntity", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/entity/Entity;getPickResult()Lnet/minecraft/world/item/ItemStack;")) - public ItemStack onPickItemFromEntity(Entity entity, Operation operation, @Local(argsOnly = true) ServerboundPickItemFromEntityPacket packet) { + public ItemStack onPickItemFromEntity(Entity entity, Operation operation, @Local(argsOnly = true, name = "packet") ServerboundPickItemFromEntityPacket packet) { ItemStack stack = PlayerPickItemEvents.ENTITY.invoker().onPickItemFromEntity(player, entity, packet.includeData()); if (stack == null) { diff --git a/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerPlayerGameModeMixin.java b/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerPlayerGameModeMixin.java index 8bec43d30fd..d74eb4251ea 100644 --- a/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerPlayerGameModeMixin.java +++ b/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerPlayerGameModeMixin.java @@ -104,7 +104,7 @@ public void interactItem(ServerPlayer player, Level level, ItemStack stack, Inte } @Inject(at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/Block;playerWillDestroy(Lnet/minecraft/world/level/Level;Lnet/minecraft/core/BlockPos;Lnet/minecraft/world/level/block/state/BlockState;Lnet/minecraft/world/entity/player/Player;)Lnet/minecraft/world/level/block/state/BlockState;"), method = "destroyBlock", cancellable = true) - private void breakBlock(BlockPos pos, CallbackInfoReturnable cir, @Local BlockEntity entity, @Local BlockState state) { + private void breakBlock(BlockPos pos, CallbackInfoReturnable cir, @Local(name = "blockEntity") BlockEntity entity, @Local(name = "state") BlockState state) { boolean result = PlayerBlockBreakEvents.BEFORE.invoker().beforeBlockBreak(this.level, this.player, pos, state, entity); if (!result) { @@ -115,7 +115,7 @@ private void breakBlock(BlockPos pos, CallbackInfoReturnable cir, @Loca } @Inject(at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/Block;destroy(Lnet/minecraft/world/level/LevelAccessor;Lnet/minecraft/core/BlockPos;Lnet/minecraft/world/level/block/state/BlockState;)V"), method = "destroyBlock") - private void onBlockBroken(BlockPos pos, CallbackInfoReturnable cir, @Local BlockEntity entity, @Local BlockState state) { + private void onBlockBroken(BlockPos pos, CallbackInfoReturnable cir, @Local(name = "blockEntity") BlockEntity entity, @Local(name = "adjustedState") BlockState state) { PlayerBlockBreakEvents.AFTER.invoker().afterBlockBreak(this.level, this.player, pos, state, entity); } } diff --git a/fabric-gametest-api-v1/src/main/java/net/fabricmc/fabric/mixin/gametest/StructureTemplateManagerMixin.java b/fabric-gametest-api-v1/src/main/java/net/fabricmc/fabric/mixin/gametest/StructureTemplateManagerMixin.java index 81c4bfee101..ba7b95a4450 100644 --- a/fabric-gametest-api-v1/src/main/java/net/fabricmc/fabric/mixin/gametest/StructureTemplateManagerMixin.java +++ b/fabric-gametest-api-v1/src/main/java/net/fabricmc/fabric/mixin/gametest/StructureTemplateManagerMixin.java @@ -79,7 +79,7 @@ private Stream streamTemplatesFromResource() { } @Inject(method = "", at = @At(value = "INVOKE", target = "Lcom/google/common/collect/ImmutableList$Builder;add(Ljava/lang/Object;)Lcom/google/common/collect/ImmutableList$Builder;", ordinal = 2, shift = At.Shift.AFTER)) - private void addFabricTemplateProvider(ResourceManager resourceManager, LevelStorageSource.LevelStorageAccess storageAccess, DataFixer dataFixer, HolderGetter blockLookup, CallbackInfo ci, @Local ImmutableList.Builder builder) { + private void addFabricTemplateProvider(ResourceManager resourceManager, LevelStorageSource.LevelStorageAccess storageAccess, DataFixer dataFixer, HolderGetter blockLookup, CallbackInfo ci, @Local(name = "builder") ImmutableList.Builder builder) { builder.add(new StructureTemplateManager.Source(this::fabric_loadSnbtFromResource, this::streamTemplatesFromResource)); } } diff --git a/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/AnvilMenuMixin.java b/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/AnvilMenuMixin.java index f339a4a2032..c34200de53f 100644 --- a/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/AnvilMenuMixin.java +++ b/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/AnvilMenuMixin.java @@ -47,7 +47,7 @@ abstract class AnvilMenuMixin extends ItemCombinerMenu { target = "Lnet/minecraft/world/item/enchantment/Enchantment;canEnchant(Lnet/minecraft/world/item/ItemStack;)Z" ) ) - private boolean callAllowEnchantingEvent(Enchantment instance, ItemStack stack, @Local Holder holder) { + private boolean callAllowEnchantingEvent(Enchantment instance, ItemStack stack, @Local(name = "enchantmentHolder") Holder holder) { return stack.canBeEnchantedWith(holder, EnchantingContext.ACCEPTABLE); } } diff --git a/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/ItemStackMixin.java b/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/ItemStackMixin.java index 87173471625..d67b2de3be4 100644 --- a/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/ItemStackMixin.java +++ b/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/ItemStackMixin.java @@ -64,7 +64,7 @@ public abstract class ItemStackMixin implements FabricItemStack { public abstract void shrink(int i); @WrapOperation(method = "hurtAndBreak(ILnet/minecraft/world/entity/LivingEntity;Lnet/minecraft/world/entity/EquipmentSlot;)V", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/item/ItemStack;hurtAndBreak(ILnet/minecraft/server/level/ServerLevel;Lnet/minecraft/server/level/ServerPlayer;Ljava/util/function/Consumer;)V")) - private void hookDamage(ItemStack instance, int amount, ServerLevel serverLevel, ServerPlayer serverPlayer, Consumer consumer, Operation original, @Local(argsOnly = true) LivingEntity entity, @Local(argsOnly = true) EquipmentSlot slot) { + private void hookDamage(ItemStack instance, int amount, ServerLevel serverLevel, ServerPlayer serverPlayer, Consumer consumer, Operation original, @Local(argsOnly = true, name = "owner") LivingEntity entity, @Local(argsOnly = true, name = "slot") EquipmentSlot slot) { CustomDamageHandler handler = ((ItemExtensions) getItem()).fabric_getCustomDamageHandler(); /* @@ -94,10 +94,10 @@ The other damage method (which original.call discards) handles the creative mode @ModifyArg(method = "addDetailsToTooltip", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/item/ItemStack;addToTooltip(Lnet/minecraft/core/component/DataComponentType;Lnet/minecraft/world/item/Item$TooltipContext;Lnet/minecraft/world/item/component/TooltipDisplay;Ljava/util/function/Consumer;Lnet/minecraft/world/item/TooltipFlag;)V")) private DataComponentType preAppendComponentTooltip( DataComponentType componentType, - @Local(argsOnly = true) Item.TooltipContext context, - @Local(argsOnly = true) TooltipDisplay displayComponent, - @Local(argsOnly = true) TooltipFlag type, - @Local(argsOnly = true) Consumer componentConsumer, + @Local(argsOnly = true, name = "context") Item.TooltipContext context, + @Local(argsOnly = true, name = "display") TooltipDisplay displayComponent, + @Local(argsOnly = true, name = "tooltipFlag") TooltipFlag type, + @Local(argsOnly = true, name = "builder") Consumer componentConsumer, @Share("index") LocalIntRef index ) { preAppendTooltip(componentType, context, displayComponent, componentConsumer, type, index); @@ -107,10 +107,10 @@ private DataComponentType preAppendComponentTooltip( @ModifyArg(method = "addDetailsToTooltip", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/item/component/TooltipDisplay;shows(Lnet/minecraft/core/component/DataComponentType;)Z")) private DataComponentType preShouldDisplay( DataComponentType componentType, - @Local(argsOnly = true) Item.TooltipContext context, - @Local(argsOnly = true) TooltipDisplay displayComponent, - @Local(argsOnly = true) TooltipFlag type, - @Local(argsOnly = true) Consumer componentConsumer, + @Local(argsOnly = true, name = "context") Item.TooltipContext context, + @Local(argsOnly = true, name = "display") TooltipDisplay displayComponent, + @Local(argsOnly = true, name = "tooltipFlag") TooltipFlag type, + @Local(argsOnly = true, name = "builder") Consumer componentConsumer, @Share("index") LocalIntRef index ) { preAppendTooltip(componentType, context, displayComponent, componentConsumer, type, index); diff --git a/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/LevelChunkMixin.java b/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/LevelChunkMixin.java index 68a621a2302..c33d8bb8d45 100644 --- a/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/LevelChunkMixin.java +++ b/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/LevelChunkMixin.java @@ -62,7 +62,7 @@ private V onLoadBlockEntity(V removedBlockEntity, BlockEntity blockEntity) { } @Inject(method = "setBlockEntity", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/entity/BlockEntity;setRemoved()V", shift = At.Shift.AFTER)) - private void onRemoveBlockEntity(BlockEntity blockEntity, CallbackInfo info, @Local(ordinal = 1) BlockEntity removedBlockEntity) { + private void onRemoveBlockEntity(BlockEntity blockEntity, CallbackInfo info, @Local(name = "previousEntry") BlockEntity removedBlockEntity) { if (removedBlockEntity != null) { if (this.getLevel() instanceof ServerLevel) { ServerBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removedBlockEntity, (ServerLevel) this.getLevel()); @@ -90,7 +90,7 @@ private Object onRemoveBlockEntity(Map map, K key) { } @Inject(method = "removeBlockEntity", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/entity/BlockEntity;setRemoved()V")) - private void onRemoveBlockEntity(BlockPos pos, CallbackInfo ci, @Local @Nullable BlockEntity removed) { + private void onRemoveBlockEntity(BlockPos pos, CallbackInfo ci, @Local(name = "removeThis") @Nullable BlockEntity removed) { if (removed != null) { if (this.getLevel() instanceof ServerLevel) { ServerBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removed, (ServerLevel) this.getLevel()); diff --git a/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/ChunkMapMixin.java b/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/ChunkMapMixin.java index dcfb556ebfd..02abade4e78 100644 --- a/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/ChunkMapMixin.java +++ b/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/ChunkMapMixin.java @@ -45,7 +45,7 @@ public abstract class ChunkMapMixin { * We inject just after "setLoaded" is made false, since here the LevelChunk is guaranteed to be unloaded. */ @Inject(method = "lambda$scheduleUnload$0", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/level/ChunkMap;save(Lnet/minecraft/world/level/chunk/ChunkAccess;)Z")) - private void onChunkUnload(ChunkHolder chunkHolder, CompletableFuture completableFuture, long l, CallbackInfo ci, @Local ChunkAccess chunk) { + private void onChunkUnload(ChunkHolder chunkHolder, CompletableFuture completableFuture, long l, CallbackInfo ci, @Local(name = "chunk") ChunkAccess chunk) { if (chunk instanceof LevelChunk levelChunk) { ServerChunkEvents.CHUNK_UNLOAD.invoker().onChunkUnload(this.level, levelChunk); } diff --git a/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/LivingEntityMixin.java b/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/LivingEntityMixin.java index 87671ffc3b2..64f9fdcb77a 100644 --- a/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/LivingEntityMixin.java +++ b/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/LivingEntityMixin.java @@ -34,7 +34,7 @@ @Mixin(LivingEntity.class) public abstract class LivingEntityMixin { @Inject(method = "collectEquipmentChanges", at = @At(value = "INVOKE", target = "Ljava/util/Map;put(Ljava/lang/Object;Ljava/lang/Object;)Ljava/lang/Object;")) - private void getEquipmentChanges(CallbackInfoReturnable<@Nullable Map> cir, @Local EquipmentSlot equipmentSlot, @Local(ordinal = 0) ItemStack previousStack, @Local(ordinal = 1) ItemStack currentStack) { + private void getEquipmentChanges(CallbackInfoReturnable<@Nullable Map> cir, @Local(name = "slot") EquipmentSlot equipmentSlot, @Local(name = "previous") ItemStack previousStack, @Local(name = "current") ItemStack currentStack) { ServerEntityEvents.EQUIPMENT_CHANGE.invoker().onChange((LivingEntity) (Object) this, equipmentSlot, previousStack, currentStack); } } diff --git a/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/MinecraftServerMixin.java b/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/MinecraftServerMixin.java index 37c2042c919..297f3983192 100644 --- a/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/MinecraftServerMixin.java +++ b/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/MinecraftServerMixin.java @@ -82,7 +82,7 @@ private V onLoadWorld(Map levels, K dimension, V level, Operation V onLoadBlockEntity(V removedBlockEntity, BlockEntity blockEntity) { } @Inject(method = "setBlockEntity", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/entity/BlockEntity;setRemoved()V", shift = At.Shift.AFTER)) - private void onRemoveBlockEntity(BlockEntity blockEntity, CallbackInfo info, @Local(ordinal = 1) BlockEntity removedBlockEntity) { + private void onRemoveBlockEntity(BlockEntity blockEntity, CallbackInfo info, @Local(name = "previousEntry") BlockEntity removedBlockEntity) { if (this.getLevel() instanceof ServerLevel) { ServerBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removedBlockEntity, (ServerLevel) this.getLevel()); } @@ -83,7 +83,7 @@ private Object onRemoveBlockEntity(Map map, K key) { } @Inject(method = "removeBlockEntity", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/entity/BlockEntity;setRemoved()V")) - private void onRemoveBlockEntity(BlockPos pos, CallbackInfo ci, @Local @Nullable BlockEntity removed) { + private void onRemoveBlockEntity(BlockPos pos, CallbackInfo ci, @Local(name = "removeThis") @Nullable BlockEntity removed) { if (removed != null && this.getLevel() instanceof ServerLevel) { ServerBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removed, (ServerLevel) this.getLevel()); } diff --git a/fabric-loot-api-v3/src/main/java/net/fabricmc/fabric/mixin/loot/ReloadableServerRegistriesMixin.java b/fabric-loot-api-v3/src/main/java/net/fabricmc/fabric/mixin/loot/ReloadableServerRegistriesMixin.java index d6f9a68bb73..cf3ed1d39fb 100644 --- a/fabric-loot-api-v3/src/main/java/net/fabricmc/fabric/mixin/loot/ReloadableServerRegistriesMixin.java +++ b/fabric-loot-api-v3/src/main/java/net/fabricmc/fabric/mixin/loot/ReloadableServerRegistriesMixin.java @@ -74,7 +74,7 @@ private static RegistryOps storeOps(HolderLookup.Provider holder, D } @WrapOperation(method = "reload", at = @At(value = "INVOKE", target = "Ljava/util/concurrent/CompletableFuture;thenApplyAsync(Ljava/util/function/Function;Ljava/util/concurrent/Executor;)Ljava/util/concurrent/CompletableFuture;")) - private static CompletableFuture> removeOps(CompletableFuture>> future, Function>, ? extends LayeredRegistryAccess> fn, Executor executor, Operation>> original, @Local RegistryOps ops) { + private static CompletableFuture> removeOps(CompletableFuture>> future, Function>, ? extends LayeredRegistryAccess> fn, Executor executor, Operation>> original, @Local(name = "ops") RegistryOps ops) { return original.call(future.thenApply(v -> { WRAPPERS.remove(ops); return v; @@ -82,7 +82,7 @@ private static CompletableFuture> removeOps } @Inject(method = "lambda$scheduleRegistryLoad$0", at = @At(value = "INVOKE", target = "Ljava/util/Map;forEach(Ljava/util/function/BiConsumer;)V")) - private static void modifyLootTable(LootDataType lootDataType, ResourceManager resourceManager, RegistryOps registryOps, CallbackInfoReturnable> cir, @Local Map map) { + private static void modifyLootTable(LootDataType lootDataType, ResourceManager resourceManager, RegistryOps registryOps, CallbackInfoReturnable> cir, @Local(name = "elements") Map map) { map.replaceAll((identifier, t) -> modifyLootTable(t, identifier, registryOps)); } diff --git a/fabric-menu-api-v1/src/main/java/net/fabricmc/fabric/mixin/menu/ServerPlayerMixin.java b/fabric-menu-api-v1/src/main/java/net/fabricmc/fabric/mixin/menu/ServerPlayerMixin.java index 93dc8908ed3..6d7c0069382 100644 --- a/fabric-menu-api-v1/src/main/java/net/fabricmc/fabric/mixin/menu/ServerPlayerMixin.java +++ b/fabric-menu-api-v1/src/main/java/net/fabricmc/fabric/mixin/menu/ServerPlayerMixin.java @@ -66,7 +66,7 @@ private void fabric_closeContainerScreenIfAllowed(ServerPlayer player, MenuProvi } @Inject(method = "openMenu(Lnet/minecraft/world/MenuProvider;)Ljava/util/OptionalInt;", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/network/ServerGamePacketListenerImpl;send(Lnet/minecraft/network/protocol/Packet;)V")) - private void fabric_storeOpenedMenu(MenuProvider factory, CallbackInfoReturnable info, @Local AbstractContainerMenu menu) { + private void fabric_storeOpenedMenu(MenuProvider factory, CallbackInfoReturnable info, @Local(name = "menu") AbstractContainerMenu menu) { if (factory instanceof ExtendedMenuProvider || (factory instanceof SimpleMenuProvider simpleFactory && simpleFactory.menuConstructor instanceof ExtendedMenuProvider)) { // Set the menu, so the factory method can access it through the player. containerMenu = menu; diff --git a/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ChatListenerMixin.java b/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ChatListenerMixin.java index 458ea6c4932..ab16607a9c0 100644 --- a/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ChatListenerMixin.java +++ b/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ChatListenerMixin.java @@ -68,7 +68,7 @@ private void fabric_onChatMessage(Component message, @Nullable PlayerChatMessage } @Inject(method = "handleSystemMessage", at = @At("HEAD"), cancellable = true) - private void fabric_allowGameMessage(Component _message, boolean overlay, CallbackInfo ci, @Local(argsOnly = true) LocalRef message) { + private void fabric_allowGameMessage(Component _message, boolean overlay, CallbackInfo ci, @Local(argsOnly = true, name = "message") LocalRef message) { if (ClientReceiveMessageEvents.ALLOW_GAME.invoker().allowReceiveGameMessage(message.get(), overlay)) { message.set(ClientReceiveMessageEvents.MODIFY_GAME.invoker().modifyReceivedGameMessage(message.get(), overlay)); ClientReceiveMessageEvents.GAME.invoker().onReceiveGameMessage(message.get(), overlay); diff --git a/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ClientPacketListenerMixin.java b/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ClientPacketListenerMixin.java index 4a24de184e5..69b56f0766b 100644 --- a/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ClientPacketListenerMixin.java +++ b/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ClientPacketListenerMixin.java @@ -34,7 +34,7 @@ @Mixin(value = ClientPacketListener.class, priority = 800) public abstract class ClientPacketListenerMixin { @Inject(method = "sendChat", at = @At("HEAD"), cancellable = true) - private void fabric_allowSendChatMessage(String _content, CallbackInfo ci, @Local(argsOnly = true) LocalRef content) { + private void fabric_allowSendChatMessage(String _content, CallbackInfo ci, @Local(argsOnly = true, name = "content") LocalRef content) { if (ClientSendMessageEvents.ALLOW_CHAT.invoker().allowSendChatMessage(content.get())) { content.set(ClientSendMessageEvents.MODIFY_CHAT.invoker().modifySendChatMessage(content.get())); ClientSendMessageEvents.CHAT.invoker().onSendChatMessage(content.get()); @@ -45,7 +45,7 @@ private void fabric_allowSendChatMessage(String _content, CallbackInfo ci, @Loca } @Inject(method = "sendCommand", at = @At("HEAD"), cancellable = true) - private void fabric_allowSendCommandMessage(String _command, CallbackInfo ci, @Local(argsOnly = true) LocalRef command) { + private void fabric_allowSendCommandMessage(String _command, CallbackInfo ci, @Local(argsOnly = true, name = "command") LocalRef command) { if (ClientSendMessageEvents.ALLOW_COMMAND.invoker().allowSendCommandMessage(command.get())) { command.set(ClientSendMessageEvents.MODIFY_COMMAND.invoker().modifySendCommandMessage(command.get())); ClientSendMessageEvents.COMMAND.invoker().onSendCommandMessage(command.get()); diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/ModelManagerMixin.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/ModelManagerMixin.java index a005305214f..a5cc8766a99 100644 --- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/ModelManagerMixin.java +++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/ModelManagerMixin.java @@ -119,7 +119,7 @@ private Function withModelDispatcher(Function function) { @Inject(method = "discoverModelDependencies", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/resources/model/ModelDiscovery;resolve()Ljava/util/Map;")) private static void resolveExtraModels( Map modelMap, BlockStateModelLoader.LoadedModels stateDefinition, ClientItemInfoLoader.LoadedClientInfos result, CallbackInfoReturnable cir, - @Local ModelDiscovery collector + @Local(name = "result") ModelDiscovery collector ) { // We know eventDispatcherFuture is available, as it is required by the item and block models (hookModels). ModelLoadingEventDispatcher eventDispatcher = ModelLoadingEventDispatcher.CURRENT.get(); @@ -127,7 +127,7 @@ private static void resolveExtraModels( } @Inject(method = "apply", at = @At(value = "RETURN")) - private void onReturnUpload(CallbackInfo ci, @Local ModelBakery.BakingResult bakedModels) { + private void onReturnUpload(CallbackInfo ci, @Local(name = "bakedModels") ModelBakery.BakingResult bakedModels) { extraModels = ((BakedModelsHooks) (Object) bakedModels).fabric_getExtraModels(); } @@ -144,7 +144,7 @@ private static BlockModel cancelVanillaDeserialize(Reader reader) { // code doesn't explicitly cast the model to BlockModel, and the enclosing method returns UnbakedModels per // its return type, it's safe to return an UnbakedModel here. @ModifyArg(method = "lambda$loadBlockModels$2(Ljava/util/Map$Entry;)Lcom/mojang/datafixers/util/Pair;", at = @At(value = "INVOKE", target = "Lcom/mojang/datafixers/util/Pair;of(Ljava/lang/Object;Ljava/lang/Object;)Lcom/mojang/datafixers/util/Pair;"), index = 1) - private static Object actuallyDeserializeModel(Object originalModel, @Local Reader reader) { + private static Object actuallyDeserializeModel(Object originalModel, @Local(name = "reader") Reader reader) { return UnbakedModelDeserializer.deserialize(reader); } } diff --git a/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/BundlePacketMixin.java b/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/BundlePacketMixin.java index 6f32e9f870d..9427181b67f 100644 --- a/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/BundlePacketMixin.java +++ b/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/BundlePacketMixin.java @@ -29,7 +29,7 @@ @Mixin(BundlePacket.class) public class BundlePacketMixin { - @ModifyVariable(method = "", at = @At("HEAD"), argsOnly = true) + @ModifyVariable(method = "", at = @At("HEAD"), argsOnly = true, name = "packets") private static Iterable> flattenBundlePackets(Iterable> value) { var packets = new ArrayList>(); iterateBundle(value, packets); diff --git a/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/ConnectionMixin.java b/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/ConnectionMixin.java index 9549e05ebe7..4a6a09d2dfe 100644 --- a/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/ConnectionMixin.java +++ b/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/ConnectionMixin.java @@ -94,7 +94,7 @@ private void disconnectAddon(CallbackInfo ci) { } @ModifyArg(method = "setupInboundProtocol", at = @At(value = "INVOKE", target = "Lio/netty/channel/Channel;writeAndFlush(Ljava/lang/Object;)Lio/netty/channel/ChannelFuture;")) - private Object injectFabricPacketSlitterHandlerInbound(Object transitioner, @Local(argsOnly = true) ProtocolInfo protocolInfo) { + private Object injectFabricPacketSlitterHandlerInbound(Object transitioner, @Local(argsOnly = true, name = "protocol") ProtocolInfo protocolInfo) { PayloadTypeRegistryImpl payloadTypeRegistry = PayloadTypeRegistryImpl.get(protocolInfo); if (payloadTypeRegistry == null) { @@ -108,7 +108,7 @@ private Object injectFabricPacketSlitterHandlerInbound(Object transitioner, @Loc } @ModifyArg(method = "setupOutboundProtocol", at = @At(value = "INVOKE", target = "Lio/netty/channel/Channel;writeAndFlush(Ljava/lang/Object;)Lio/netty/channel/ChannelFuture;")) - private Object injectFabricPacketSlitterHandlerOutbound(Object transitioner, @Local(argsOnly = true) ProtocolInfo protocolInfo) { + private Object injectFabricPacketSlitterHandlerOutbound(Object transitioner, @Local(argsOnly = true, name = "protocol") ProtocolInfo protocolInfo) { PayloadTypeRegistryImpl payloadTypeRegistry = PayloadTypeRegistryImpl.get(protocolInfo); if (payloadTypeRegistry == null) { diff --git a/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/IdDispatchCodecMixin.java b/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/IdDispatchCodecMixin.java index df152cb818a..67cf556330a 100644 --- a/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/IdDispatchCodecMixin.java +++ b/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/IdDispatchCodecMixin.java @@ -34,7 +34,7 @@ public abstract class IdDispatchCodecMixin implements StreamCodec { // Add the custom payload id to the error message @Inject(method = "encode(Lio/netty/buffer/ByteBuf;Ljava/lang/Object;)V", at = @At(value = "NEW", target = "(Ljava/lang/String;Ljava/lang/Throwable;)Lio/netty/handler/codec/EncoderException;")) - public void encode(B byteBuf, V packet, CallbackInfo ci, @Local(ordinal = 1) T packetType, @Local Exception e) { + public void encode(B byteBuf, V packet, CallbackInfo ci, @Local(name = "type") T packetType, @Local(name = "e") Exception e) { CustomPacketPayload payload = null; if (packet instanceof ServerboundCustomPayloadPacket customPayloadPacket) { diff --git a/fabric-object-builder-api-v1/src/main/java/net/fabricmc/fabric/mixin/object/builder/EntityTypeBuilderMixin.java b/fabric-object-builder-api-v1/src/main/java/net/fabricmc/fabric/mixin/object/builder/EntityTypeBuilderMixin.java index ffd4da2c2fc..97fdf7e7799 100644 --- a/fabric-object-builder-api-v1/src/main/java/net/fabricmc/fabric/mixin/object/builder/EntityTypeBuilderMixin.java +++ b/fabric-object-builder-api-v1/src/main/java/net/fabricmc/fabric/mixin/object/builder/EntityTypeBuilderMixin.java @@ -100,7 +100,7 @@ private static EntityType castMob( } @WrapOperation(method = "build", at = @At(value = "INVOKE", target = "Lnet/minecraft/util/Util;fetchChoiceType(Lcom/mojang/datafixers/DSL$TypeReference;Ljava/lang/String;)Lcom/mojang/datafixers/types/Type;")) - private @Nullable Type allowNoModdedDatafixers(DSL.TypeReference typeReference, String id, Operation> original, @Local(argsOnly = true) ResourceKey> resourceKey) { + private @Nullable Type allowNoModdedDatafixers(DSL.TypeReference typeReference, String id, Operation> original, @Local(argsOnly = true, name = "name") ResourceKey> resourceKey) { if (!resourceKey.identifier().getNamespace().equals(Identifier.DEFAULT_NAMESPACE)) { // Don't try to resolve the choice type for modded entities. return null; diff --git a/fabric-particles-v1/src/client/java/net/fabricmc/fabric/mixin/client/particle/TerrainParticleMixin.java b/fabric-particles-v1/src/client/java/net/fabricmc/fabric/mixin/client/particle/TerrainParticleMixin.java index 1e502a4c618..b84780c700b 100644 --- a/fabric-particles-v1/src/client/java/net/fabricmc/fabric/mixin/client/particle/TerrainParticleMixin.java +++ b/fabric-particles-v1/src/client/java/net/fabricmc/fabric/mixin/client/particle/TerrainParticleMixin.java @@ -55,9 +55,10 @@ private TerrainParticleMixin() { from = @At(value = "FIELD", target = "Lnet/minecraft/client/particle/TerrainParticle;bCol:F", ordinal = 0, opcode = Opcodes.PUTFIELD), to = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/state/BlockState;is(Ljava/lang/Object;)Z") ), - allow = 1 + allow = 1, + name = "blockState" ) - private BlockState removeUntintableParticles(BlockState state, @Local(argsOnly = true) ClientLevel level, @Local(argsOnly = true) BlockPos blockPos) { + private BlockState removeUntintableParticles(BlockState state, @Local(argsOnly = true, name = "level") ClientLevel level, @Local(argsOnly = true, name = "pos") BlockPos blockPos) { if (!ParticleRenderEvents.ALLOW_TERRAIN_PARTICLE_TINT.invoker().allowTerrainParticleTint(state, level, blockPos)) { // As of 1.20.1, vanilla hardcodes grass block particles to not get tinted. return Blocks.GRASS_BLOCK.defaultBlockState(); diff --git a/fabric-particles-v1/src/main/java/net/fabricmc/fabric/mixin/particle/EntityMixin.java b/fabric-particles-v1/src/main/java/net/fabricmc/fabric/mixin/particle/EntityMixin.java index e87e624e56a..b026366ebce 100644 --- a/fabric-particles-v1/src/main/java/net/fabricmc/fabric/mixin/particle/EntityMixin.java +++ b/fabric-particles-v1/src/main/java/net/fabricmc/fabric/mixin/particle/EntityMixin.java @@ -30,7 +30,7 @@ @Mixin(Entity.class) abstract class EntityMixin { @ModifyExpressionValue(method = "spawnSprintParticle", at = @At(value = "NEW", target = "(Lnet/minecraft/core/particles/ParticleType;Lnet/minecraft/world/level/block/state/BlockState;)Lnet/minecraft/core/particles/BlockParticleOption;")) - private BlockParticleOption modifyBlockStateParticleOption(BlockParticleOption original, @Local(ordinal = 0) BlockPos blockPos) { + private BlockParticleOption modifyBlockStateParticleOption(BlockParticleOption original, @Local(name = "pos") BlockPos blockPos) { ((BlockParticleOptionExtension) original).fabric_setBlockPos(blockPos); return original; } diff --git a/fabric-recipe-api-v1/src/main/java/net/fabricmc/fabric/mixin/recipe/sync/RecipeMapMixin.java b/fabric-recipe-api-v1/src/main/java/net/fabricmc/fabric/mixin/recipe/sync/RecipeMapMixin.java index 18f771123ca..651d34af5c6 100644 --- a/fabric-recipe-api-v1/src/main/java/net/fabricmc/fabric/mixin/recipe/sync/RecipeMapMixin.java +++ b/fabric-recipe-api-v1/src/main/java/net/fabricmc/fabric/mixin/recipe/sync/RecipeMapMixin.java @@ -57,7 +57,7 @@ private static void provideSerializerMap(Iterable> recipes, Call } @Inject(method = "create", at = @At(value = "INVOKE", target = "Lcom/google/common/collect/ImmutableMap$Builder;put(Ljava/lang/Object;Ljava/lang/Object;)Lcom/google/common/collect/ImmutableMap$Builder;")) - private static void fillSerializerMap(Iterable> recipes, CallbackInfoReturnable cir, @Local RecipeHolder entry, + private static void fillSerializerMap(Iterable> recipes, CallbackInfoReturnable cir, @Local(name = "recipe") RecipeHolder entry, @Share("bySerializer") LocalRef, List>>> bySerializer) { List> list = bySerializer.get().get(entry.value().getSerializer()); diff --git a/fabric-registry-sync-v0/src/main/java/net/fabricmc/fabric/mixin/registry/sync/RegistriesMixin.java b/fabric-registry-sync-v0/src/main/java/net/fabricmc/fabric/mixin/registry/sync/RegistriesMixin.java index 9bc5bf5c407..65ecfac7df1 100644 --- a/fabric-registry-sync-v0/src/main/java/net/fabricmc/fabric/mixin/registry/sync/RegistriesMixin.java +++ b/fabric-registry-sync-v0/src/main/java/net/fabricmc/fabric/mixin/registry/sync/RegistriesMixin.java @@ -31,7 +31,7 @@ @Mixin(Registries.class) public class RegistriesMixin { @ModifyReturnValue(method = "elementsDirPath", at = @At("RETURN")) - private static String prependDirectoryWithNamespace(String original, @Local(argsOnly = true) ResourceKey> registryRef) { + private static String prependDirectoryWithNamespace(String original, @Local(argsOnly = true, name = "registryKey") ResourceKey> registryRef) { Identifier id = registryRef.identifier(); if (!id.getNamespace().equals(Identifier.DEFAULT_NAMESPACE)) { @@ -42,7 +42,7 @@ private static String prependDirectoryWithNamespace(String original, @Local(args } @ModifyReturnValue(method = "tagsDirPath", at = @At("RETURN")) - private static String prependTagDirectoryWithNamespace(String original, @Local(argsOnly = true) ResourceKey> registryRef) { + private static String prependTagDirectoryWithNamespace(String original, @Local(argsOnly = true, name = "registryKey") ResourceKey> registryRef) { Identifier id = registryRef.identifier(); if (!id.getNamespace().equals(Identifier.DEFAULT_NAMESPACE)) { diff --git a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/model/SimpleModelWrapperMixin.java b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/model/SimpleModelWrapperMixin.java index 6ca0c412906..63729feec8d 100644 --- a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/model/SimpleModelWrapperMixin.java +++ b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/model/SimpleModelWrapperMixin.java @@ -54,7 +54,7 @@ abstract class SimpleModelWrapperMixin implements BlockModelPart { private boolean useAmbientOcclusion; @Inject(method = "bake(Lnet/minecraft/client/resources/model/ModelBaker;Lnet/minecraft/resources/Identifier;Lnet/minecraft/client/resources/model/ModelState;)Lnet/minecraft/client/renderer/block/model/BlockModelPart;", at = @At(value = "INVOKE", target = "net/minecraft/client/resources/model/QuadCollection.getAll()Ljava/util/List;")) - private static void validateMeshAtlas(final ModelBaker modelBakery, final Identifier location, final ModelState state, CallbackInfoReturnable cir, @Local QuadCollection geometry, @Local LocalRef> forbiddenSpritesRef) { + private static void validateMeshAtlas(final ModelBaker modelBakery, final Identifier location, final ModelState state, CallbackInfoReturnable cir, @Local(name = "geometry") QuadCollection geometry, @Local(name = "forbiddenSprites") LocalRef> forbiddenSpritesRef) { if (geometry instanceof MeshQuadCollection meshGeometry) { meshGeometry.getMesh().forEach(quad -> { if (quad.atlas() != QuadAtlas.BLOCK) { diff --git a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/particle/ScreenEffectRendererMixin.java b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/particle/ScreenEffectRendererMixin.java index f8f62de9b78..e3a79878cd5 100644 --- a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/particle/ScreenEffectRendererMixin.java +++ b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/particle/ScreenEffectRendererMixin.java @@ -39,7 +39,7 @@ abstract class ScreenEffectRendererMixin { private static BlockPos pos; @Redirect(method = "renderScreenEffect", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/renderer/block/BlockModelShaper;getParticleIcon(Lnet/minecraft/world/level/block/state/BlockState;)Lnet/minecraft/client/renderer/texture/TextureAtlasSprite;")) - private static TextureAtlasSprite getParticleIconProxy(BlockModelShaper models, BlockState state, @Local Player player) { + private static TextureAtlasSprite getParticleIconProxy(BlockModelShaper models, BlockState state, @Local(name = "player") Player player) { if (pos != null) { TextureAtlasSprite sprite = models.getParticleIcon(state, player.level(), pos); pos = null; @@ -50,7 +50,7 @@ private static TextureAtlasSprite getParticleIconProxy(BlockModelShaper models, } @Inject(method = "getViewBlockingState", at = @At("RETURN")) - private static void onReturnGetInWallBlockState(CallbackInfoReturnable<@Nullable BlockState> cir, @Local BlockPos.MutableBlockPos mutable) { + private static void onReturnGetInWallBlockState(CallbackInfoReturnable<@Nullable BlockState> cir, @Local(name = "testPos") BlockPos.MutableBlockPos mutable) { if (cir.getReturnValue() != null) { pos = mutable.immutable(); } else { diff --git a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/ItemFrameRendererMixin.java b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/ItemFrameRendererMixin.java index 2eff3c7c6b3..a46311f1d23 100644 --- a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/ItemFrameRendererMixin.java +++ b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/ItemFrameRendererMixin.java @@ -34,7 +34,7 @@ abstract class ItemFrameRendererMixin { // Provide the BlockState as context. @Redirect(method = "submit(Lnet/minecraft/client/renderer/entity/state/ItemFrameRenderState;Lcom/mojang/blaze3d/vertex/PoseStack;Lnet/minecraft/client/renderer/SubmitNodeCollector;Lnet/minecraft/client/renderer/state/CameraRenderState;)V", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/renderer/SubmitNodeCollector;submitBlockModel(Lcom/mojang/blaze3d/vertex/PoseStack;Lnet/minecraft/client/renderer/rendertype/RenderType;Lnet/minecraft/client/renderer/block/model/BlockStateModel;FFFIII)V")) - private void renderProxy(SubmitNodeCollector submitNodeCollector, PoseStack poseStack, RenderType renderType, BlockStateModel model, float r, float g, float b, int light, int overlay, int outlineColor, @Local BlockState blockState) { + private void renderProxy(SubmitNodeCollector submitNodeCollector, PoseStack poseStack, RenderType renderType, BlockStateModel model, float r, float g, float b, int light, int overlay, int outlineColor, @Local(name = "fakeBlockState") BlockState blockState) { // The vertex consumer is for a special layer that renders solid, but vanilla has no equivalent // cutout/translucent layers that we can use here without risking compatibility. submitNodeCollector.submitBlockStateModel(poseStack, blockLayer -> renderType, model, r, g, b, light, overlay, outlineColor, EmptyBlockAndTintGetter.INSTANCE, BlockPos.ZERO, blockState); diff --git a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/item/BlockModelWrapperMixin.java b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/item/BlockModelWrapperMixin.java index c5f04f2a2e3..1197fe1469e 100644 --- a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/item/BlockModelWrapperMixin.java +++ b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/item/BlockModelWrapperMixin.java @@ -56,7 +56,7 @@ abstract class BlockModelWrapperMixin implements ItemModel, BlockModelWrapperExt private Mesh mesh; @Inject(method = "update", at = @At("RETURN")) - private void onReturnUpdate(final ItemStackRenderState output, final ItemStack item, final ItemModelResolver resolver, final ItemDisplayContext displayContext, final @Nullable ClientLevel level, final @Nullable ItemOwner owner, final int seed, CallbackInfo ci, @Local ItemStackRenderState.LayerRenderState layer) { + private void onReturnUpdate(final ItemStackRenderState output, final ItemStack item, final ItemModelResolver resolver, final ItemDisplayContext displayContext, final @Nullable ClientLevel level, final @Nullable ItemOwner owner, final int seed, CallbackInfo ci, @Local(name = "layer") ItemStackRenderState.LayerRenderState layer) { if (mesh != null) { // This logic matches that of ITEM_RENDER_TYPE_GETTER and BLOCK_RENDER_TYPE_GETTER ChunkSectionLayer defaultSectionLayer; diff --git a/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/ItemStackRenderStateMixin.java b/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/ItemStackRenderStateMixin.java index 6d20f4eac93..4cea9d9f990 100644 --- a/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/ItemStackRenderStateMixin.java +++ b/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/ItemStackRenderStateMixin.java @@ -38,12 +38,12 @@ @Mixin(ItemStackRenderState.class) abstract class ItemStackRenderStateMixin { @Inject(method = "visitExtents(Ljava/util/function/Consumer;)V", at = @At(value = "NEW", target = "com/mojang/blaze3d/vertex/PoseStack$Pose")) - private void afterInitVecLoad(Consumer posConsumer, CallbackInfo ci, @Local Vector3f vec, @Share("pipe") LocalRef pipeRef) { + private void afterInitVecLoad(Consumer posConsumer, CallbackInfo ci, @Local(name = "scratch") Vector3f vec, @Share("pipe") LocalRef pipeRef) { pipeRef.set(new QuadToPosPipe(posConsumer, vec)); } @Inject(method = "visitExtents(Ljava/util/function/Consumer;)V", at = @At(value = "INVOKE", target = "Lcom/mojang/blaze3d/vertex/PoseStack$Pose;setIdentity()V")) - private void afterLayerLoad(Consumer posConsumer, CallbackInfo ci, @Local(ordinal = 0) Vector3f vec, @Local ItemStackRenderState.LayerRenderState layer, @Local Matrix4f matrix, @Share("pipe") LocalRef pipeRef) { + private void afterLayerLoad(Consumer posConsumer, CallbackInfo ci, @Local(name = "scratch") Vector3f vec, @Local(name = "layer") ItemStackRenderState.LayerRenderState layer, @Local(name = "poseTransform") Matrix4f matrix, @Share("pipe") LocalRef pipeRef) { MutableMeshImpl mutableMesh = ((AccessLayerRenderState) layer).fabric_getMutableMesh(); if (mutableMesh.size() > 0) { diff --git a/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/SectionCompilerMixin.java b/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/SectionCompilerMixin.java index 21f78f82730..01c0686fc88 100644 --- a/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/SectionCompilerMixin.java +++ b/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/SectionCompilerMixin.java @@ -82,10 +82,10 @@ protected abstract BufferBuilder getOrBeginLayer( private void hookBuild(SectionPos sectionPos, RenderSectionRegion region, VertexSorting sorter, SectionBufferBuilderPack buffers, CallbackInfoReturnable cir, - @Local(ordinal = 0) BlockPos sectionOrigin, - @Local(ordinal = 0) PoseStack poseStack, - @Local(ordinal = 0) Map builderMap, - @Local(ordinal = 0) RandomSource random) { + @Local(name = "minPos") BlockPos sectionOrigin, + @Local(name = "poseStack") PoseStack poseStack, + @Local(name = "startedLayers") Map builderMap, + @Local(name = "random") RandomSource random) { // hook just before iterating over the render chunk's blocks to capture the buffer builder map TerrainRenderContext renderer = TerrainRenderContext.POOL.get(); renderer.prepare(region, sectionOrigin, diff --git a/fabric-rendering-fluids-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/fluid/LiquidBlockRendererMixin.java b/fabric-rendering-fluids-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/fluid/LiquidBlockRendererMixin.java index 652f097474c..097df953ca2 100644 --- a/fabric-rendering-fluids-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/fluid/LiquidBlockRendererMixin.java +++ b/fabric-rendering-fluids-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/fluid/LiquidBlockRendererMixin.java @@ -108,7 +108,7 @@ private TextureAtlasSprite getOrDefault(int index, TextureAtlasSprite original) @ModifyVariable( method = "tesselate", at = @At("STORE"), - ordinal = 0 + name = "stillSprite" ) public TextureAtlasSprite modStill(TextureAtlasSprite original) { return getOrDefault(0, original); @@ -117,7 +117,7 @@ public TextureAtlasSprite modStill(TextureAtlasSprite original) { @ModifyVariable( method = "tesselate", at = @At("STORE"), - ordinal = 1 + name = "flowingSprite" ) public TextureAtlasSprite modFlowing(TextureAtlasSprite original) { return getOrDefault(1, original); @@ -147,9 +147,9 @@ public int modTintColor(int original, BlockAndTintGetter level, BlockPos pos, Ve private TextureAtlasSprite modifyOverlaySprite( TextureAtlasSprite waterOverlay, BlockAndTintGetter level, - @Local(ordinal = 1) BlockPos neighborPos, - @Local(ordinal = 0) boolean isLava, - @Local(ordinal = 1) TextureAtlasSprite flowingSprite, + @Local(name = "tPos") BlockPos neighborPos, + @Local(name = "isLava") boolean isLava, + @Local(name = "flowingSprite") TextureAtlasSprite flowingSprite, @Share("useOverlay") LocalBooleanRef useOverlay ) { final FluidRenderHandlerInfo info = FluidRenderingImpl.getCurrentInfo(); diff --git a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/CapeLayerMixin.java b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/CapeLayerMixin.java index 5a0d4e747d5..9240ce728d2 100644 --- a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/CapeLayerMixin.java +++ b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/CapeLayerMixin.java @@ -32,7 +32,7 @@ @Mixin(CapeLayer.class) public class CapeLayerMixin { @Inject(at = @At(value = "HEAD"), method = "submit(Lcom/mojang/blaze3d/vertex/PoseStack;Lnet/minecraft/client/renderer/SubmitNodeCollector;ILnet/minecraft/client/renderer/entity/state/AvatarRenderState;FF)V", cancellable = true) - public void injectCapeRenderCheck(PoseStack poseStack, SubmitNodeCollector submitNodeCollector, int i, AvatarRenderState avatarRenderState, float f, float g, CallbackInfo ci, @Local(argsOnly = true) AvatarRenderState state) { + public void injectCapeRenderCheck(PoseStack poseStack, SubmitNodeCollector submitNodeCollector, int i, AvatarRenderState avatarRenderState, float f, float g, CallbackInfo ci, @Local(argsOnly = true, name = "state") AvatarRenderState state) { if (!LivingEntityFeatureRenderEvents.ALLOW_CAPE_RENDER.invoker().allowCapeRender(state)) { ci.cancel(); } diff --git a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/DebugOptionsScreenOptionListMixin.java b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/DebugOptionsScreenOptionListMixin.java index 39b798a6873..3e0dd466f7f 100644 --- a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/DebugOptionsScreenOptionListMixin.java +++ b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/DebugOptionsScreenOptionListMixin.java @@ -38,7 +38,7 @@ private static int sort(Identifier o1, Identifier o2) { } @WrapOperation(method = "updateSearch", at = @At(value = "INVOKE", target = "Ljava/lang/String;contains(Ljava/lang/CharSequence;)Z")) - private boolean searchPath(String instance, CharSequence searchStrings, Operation original, @Local Map.Entry entry) { + private boolean searchPath(String instance, CharSequence searchStrings, Operation original, @Local(name = "entry") Map.Entry entry) { final String namespace = entry.getKey().getNamespace(); return original.call(instance, searchStrings) || (!Identifier.DEFAULT_NAMESPACE.equals(namespace) && namespace.contains(searchStrings)); diff --git a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/EntityRenderDispatcherMixin.java b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/EntityRenderDispatcherMixin.java index e30fdf466bc..38b13fd399e 100644 --- a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/EntityRenderDispatcherMixin.java +++ b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/EntityRenderDispatcherMixin.java @@ -31,7 +31,7 @@ @Mixin(EntityRenderDispatcher.class) class EntityRenderDispatcherMixin { @Inject(method = "onResourceManagerReload", at = @At("TAIL")) - private void createArmorRenderers(ResourceManager manager, CallbackInfo ci, @Local EntityRendererProvider.Context context) { + private void createArmorRenderers(ResourceManager manager, CallbackInfo ci, @Local(name = "context") EntityRendererProvider.Context context) { ArmorRendererRegistryImpl.createArmorRenderers(context); } } diff --git a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/GuiRendererMixin.java b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/GuiRendererMixin.java index beb3f1c05de..52c22f4b760 100644 --- a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/GuiRendererMixin.java +++ b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/GuiRendererMixin.java @@ -93,7 +93,7 @@ private void postPrepareSpecialElements(CallbackInfo ci) { pipRendererPools.values().forEach(PictureInPictureRendererPool::cleanUpUnusedRenderers); } - @ModifyVariable(method = "preparePictureInPictureState", at = @At("STORE")) + @ModifyVariable(method = "preparePictureInPictureState", at = @At("STORE"), name = "renderer") private PictureInPictureRenderer substituteSpecialElementRenderer(PictureInPictureRenderer original, T elementState) { if (original == null || !hasFabricInitialized) { return original; diff --git a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LayerDefinitionsMixin.java b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LayerDefinitionsMixin.java index e0b5c1cdd92..73bc191fd28 100644 --- a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LayerDefinitionsMixin.java +++ b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LayerDefinitionsMixin.java @@ -36,7 +36,7 @@ @Mixin(LayerDefinitions.class) abstract class LayerDefinitionsMixin { @Inject(method = "createRoots", at = @At(value = "INVOKE", target = "Lcom/google/common/collect/ImmutableMap$Builder;build()Lcom/google/common/collect/ImmutableMap;")) - private static void registerExtraModelData(CallbackInfoReturnable> info, @Local ImmutableMap.Builder builder) { + private static void registerExtraModelData(CallbackInfoReturnable> info, @Local(name = "result") ImmutableMap.Builder builder) { for (Map.Entry entry : ModelLayerImpl.PROVIDERS.entrySet()) { builder.put(entry.getKey(), entry.getValue().createLayerDefinition()); } diff --git a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LivingEntityRendererMixin.java b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LivingEntityRendererMixin.java index 1b86dee224c..5cffccc4b4e 100644 --- a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LivingEntityRendererMixin.java +++ b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LivingEntityRendererMixin.java @@ -36,7 +36,7 @@ abstract class LivingEntityRendererMixin { method = "extractRenderState(Lnet/minecraft/world/entity/LivingEntity;Lnet/minecraft/client/renderer/entity/state/LivingEntityRenderState;F)V", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/renderer/entity/layers/HumanoidArmorLayer;shouldRender(Lnet/minecraft/world/item/ItemStack;Lnet/minecraft/world/entity/EquipmentSlot;)Z") ) - private boolean toggleDefaultHeadItem(ItemStack headStack, EquipmentSlot slot, Operation original, @Local(argsOnly = true) LivingEntity entity) { + private boolean toggleDefaultHeadItem(ItemStack headStack, EquipmentSlot slot, Operation original, @Local(argsOnly = true, name = "entity") LivingEntity entity) { // Return value: true if the item isn't rendered if (original.call(headStack, slot)) return true; ArmorRenderer renderer = ArmorRendererRegistryImpl.get(headStack.getItem()); diff --git a/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/PackMixin.java b/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/PackMixin.java index 608f02b4d21..54109c569e3 100644 --- a/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/PackMixin.java +++ b/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/PackMixin.java @@ -32,8 +32,8 @@ @Mixin(Pack.class) public class PackMixin { - @ModifyVariable(method = "readPackMetadata", at = @At("STORE")) - private static List applyOverlayConditions(List overlays, @Local PackResources resourcePack) throws IOException { + @ModifyVariable(method = "readPackMetadata", at = @At("STORE"), name = "overlaySet") + private static List applyOverlayConditions(List overlays, @Local(name = "pack") PackResources resourcePack) throws IOException { List appliedOverlays = new ArrayList<>(overlays); OverlayConditionsMetadata overlayMetadata = resourcePack.getMetadataSection(OverlayConditionsMetadata.SERIALIZER); diff --git a/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/RegistryLoadTaskPendingRegistrationMixin.java b/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/RegistryLoadTaskPendingRegistrationMixin.java index ac7e8ac4596..bc7b80325cf 100644 --- a/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/RegistryLoadTaskPendingRegistrationMixin.java +++ b/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/RegistryLoadTaskPendingRegistrationMixin.java @@ -35,7 +35,7 @@ @Mixin(RegistryLoadTask.PendingRegistration.class) public abstract class RegistryLoadTaskPendingRegistrationMixin { @Inject(method = "loadFromResource", at = @At(value = "INVOKE", target = "Lcom/mojang/serialization/Decoder;parse(Lcom/mojang/serialization/DynamicOps;Ljava/lang/Object;)Lcom/mojang/serialization/DataResult;"), cancellable = true) - private static void loadFromResource(Decoder elementDecoder, RegistryOps ops, ResourceKey elementKey, Resource thunk, CallbackInfoReturnable> cir, @Local JsonElement jsonElement) { + private static void loadFromResource(Decoder elementDecoder, RegistryOps ops, ResourceKey elementKey, Resource thunk, CallbackInfoReturnable> cir, @Local(name = "json") JsonElement jsonElement) { if (jsonElement.isJsonObject() && !ResourceConditionsImpl.applyResourceConditions(jsonElement.getAsJsonObject(), elementKey.registry().toString(), elementKey.identifier(), ops.lookupProvider)) { cir.setReturnValue(Either.right(ResourceConditionsImpl.DISABLED_RESOURCE_EXCEPTION)); } diff --git a/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/SimpleJsonResourceReloadListenerMixin.java b/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/SimpleJsonResourceReloadListenerMixin.java index 47005e6909c..30a0aa500b4 100644 --- a/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/SimpleJsonResourceReloadListenerMixin.java +++ b/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/SimpleJsonResourceReloadListenerMixin.java @@ -48,8 +48,8 @@ public class SimpleJsonResourceReloadListenerMixin { @WrapOperation(method = "scanDirectory(Lnet/minecraft/server/packs/resources/ResourceManager;Lnet/minecraft/resources/FileToIdConverter;Lcom/mojang/serialization/DynamicOps;Lcom/mojang/serialization/Codec;Ljava/util/Map;)V", at = @At(value = "INVOKE", target = "Lcom/mojang/serialization/Codec;parse(Lcom/mojang/serialization/DynamicOps;Ljava/lang/Object;)Lcom/mojang/serialization/DataResult;")) private static DataResult applyResourceConditions(Codec instance, DynamicOps dynamicOps, Object object, Operation> original, - @Local(argsOnly = true) FileToIdConverter resourceFinder, - @Local Map.Entry entry) { + @Local(argsOnly = true, name = "lister") FileToIdConverter resourceFinder, + @Local(name = "entry") Map.Entry entry) { final JsonElement resourceData = (JsonElement) object; RegistryOps.@Nullable RegistryInfoLookup registryInfo = null; diff --git a/fabric-resource-loader-v1/src/client/java/net/fabricmc/fabric/mixin/resource/client/CreateWorldScreenMixin.java b/fabric-resource-loader-v1/src/client/java/net/fabricmc/fabric/mixin/resource/client/CreateWorldScreenMixin.java index b6b8daf27fe..1e2d817a1d9 100644 --- a/fabric-resource-loader-v1/src/client/java/net/fabricmc/fabric/mixin/resource/client/CreateWorldScreenMixin.java +++ b/fabric-resource-loader-v1/src/client/java/net/fabricmc/fabric/mixin/resource/client/CreateWorldScreenMixin.java @@ -46,7 +46,8 @@ private CreateWorldScreenMixin() { } @ModifyVariable(method = "openCreateWorldScreen(Lnet/minecraft/client/Minecraft;Ljava/lang/Runnable;Ljava/util/function/Function;Lnet/minecraft/client/gui/screens/worldselection/WorldCreationContextMapper;Lnet/minecraft/resources/ResourceKey;Lnet/minecraft/client/gui/screens/worldselection/CreateWorldCallback;)V", - at = @At(value = "INVOKE", target = "Lnet/minecraft/client/gui/screens/worldselection/CreateWorldScreen;createDefaultLoadConfig(Lnet/minecraft/server/packs/repository/PackRepository;Lnet/minecraft/world/level/WorldDataConfiguration;)Lnet/minecraft/server/WorldLoader$InitConfig;")) + at = @At(value = "INVOKE", target = "Lnet/minecraft/client/gui/screens/worldselection/CreateWorldScreen;createDefaultLoadConfig(Lnet/minecraft/server/packs/repository/PackRepository;Lnet/minecraft/world/level/WorldDataConfiguration;)Lnet/minecraft/server/WorldLoader$InitConfig;"), + name = "vanillaOnlyPackRepository") private static PackRepository onCreateResManagerInit(PackRepository manager) { // Add mod data packs to the initial res pack manager so they are active even if the user doesn't use custom data packs manager.sources.add(new ModResourcePackCreator(PackType.SERVER_DATA)); diff --git a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/DataPackCommandMixin.java b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/DataPackCommandMixin.java index f58ea2d3e48..e156e38f321 100644 --- a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/DataPackCommandMixin.java +++ b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/DataPackCommandMixin.java @@ -62,7 +62,7 @@ private static Stream filterDisabledPackSuggestions(Stream instance, } @Inject(method = "getPack", at = @At(value = "INVOKE", target = "Ljava/util/Collection;contains(Ljava/lang/Object;)Z")) - private static void errorOnInternalPack(CommandContext context, String name, boolean enable, CallbackInfoReturnable cir, @Local Pack profile) throws CommandSyntaxException { + private static void errorOnInternalPack(CommandContext context, String name, boolean enable, CallbackInfoReturnable cir, @Local(name = "pack") Pack profile) throws CommandSyntaxException { if (((FabricPack) profile).fabric$isHidden()) throw INTERNAL_PACK_EXCEPTION.create(profile.getId()); } } diff --git a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/PackRepositoryMixin.java b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/PackRepositoryMixin.java index e9f50299ecb..c4e9366ea11 100644 --- a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/PackRepositoryMixin.java +++ b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/PackRepositoryMixin.java @@ -84,19 +84,19 @@ public void construct(RepositorySource[] resourcePackProviders, CallbackInfo inf } @Inject(method = "rebuildSelected", at = @At(value = "INVOKE", target = "Lcom/google/common/collect/ImmutableList;copyOf(Ljava/util/Collection;)Lcom/google/common/collect/ImmutableList;")) - private void handleAutoEnableDisable(Collection enabledNames, CallbackInfoReturnable> cir, @Local List enabledAfterFirstRun) { + private void handleAutoEnableDisable(Collection enabledNames, CallbackInfoReturnable> cir, @Local(name = "selectedAndPresent") List enabledAfterFirstRun) { ModPackResourcesUtil.refreshAutoEnabledPacks(enabledAfterFirstRun, this.available); } @Inject(method = "addPack", at = @At(value = "INVOKE", target = "Ljava/util/List;add(Ljava/lang/Object;)Z", shift = At.Shift.AFTER)) - private void handleAutoEnable(String profile, CallbackInfoReturnable cir, @Local List newlyEnabled) { + private void handleAutoEnable(String profile, CallbackInfoReturnable cir, @Local(name = "selectedCopy") List newlyEnabled) { if (ModResourcePackCreator.POST_CHANGE_HANDLE_REQUIRED.contains(profile)) { ModPackResourcesUtil.refreshAutoEnabledPacks(newlyEnabled, this.available); } } @Inject(method = "removePack", at = @At(value = "INVOKE", target = "Ljava/util/List;remove(Ljava/lang/Object;)Z")) - private void handleAutoDisable(String profile, CallbackInfoReturnable cir, @Local List enabled) { + private void handleAutoDisable(String profile, CallbackInfoReturnable cir, @Local(name = "selectedCopy") List enabled) { if (ModResourcePackCreator.POST_CHANGE_HANDLE_REQUIRED.contains(profile)) { Set currentlyEnabled = enabled.stream().map(Pack::getId).collect(Collectors.toSet()); enabled.removeIf(p -> !((FabricPack) p).fabric$parentsEnabled(currentlyEnabled)); diff --git a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/ReloadableServerResourcesMixin.java b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/ReloadableServerResourcesMixin.java index aea3fb93069..286ddc9db44 100644 --- a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/ReloadableServerResourcesMixin.java +++ b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/ReloadableServerResourcesMixin.java @@ -50,9 +50,9 @@ public class ReloadableServerResourcesMixin implements FabricDataResourceStoreHo ) private static List onSetupDataReloaders( List reloaders, - @Local(argsOnly = true) ReloadableServerRegistries.LoadResult loadResult, - @Local(argsOnly = true) FeatureFlagSet featureSet, - @Local ReloadableServerResources dataPackContents + @Local(argsOnly = true, name = "fullRegistries") ReloadableServerRegistries.LoadResult loadResult, + @Local(argsOnly = true, name = "enabledFeatures") FeatureFlagSet featureSet, + @Local(name = "result") ReloadableServerResources dataPackContents ) { var list = new ArrayList<>(reloaders); list.addFirst( diff --git a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/SimpleReloadInstanceMixin.java b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/SimpleReloadInstanceMixin.java index 046606046b5..1bb1ea55355 100644 --- a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/SimpleReloadInstanceMixin.java +++ b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/SimpleReloadInstanceMixin.java @@ -40,7 +40,7 @@ public class SimpleReloadInstanceMixin { target = "Lnet/minecraft/server/packs/resources/SimpleReloadInstance;of(Lnet/minecraft/server/packs/resources/ResourceManager;Ljava/util/List;Ljava/util/concurrent/Executor;Ljava/util/concurrent/Executor;Ljava/util/concurrent/CompletableFuture;)Lnet/minecraft/server/packs/resources/ReloadInstance;" ) ) - private static List sortSimple(List reloaders, @Local(argsOnly = true) ResourceManager resourceManager) { + private static List sortSimple(List reloaders, @Local(argsOnly = true, name = "resourceManager") ResourceManager resourceManager) { if (resourceManager instanceof FabricMultiPackResourceManager flrm) { return ResourceLoaderImpl.sort(flrm.fabric$getPackType(), reloaders); } @@ -55,7 +55,7 @@ private static List sortSimple(List sortProfiled(List reloaders, @Local(argsOnly = true) ResourceManager resourceManager) { + private static List sortProfiled(List reloaders, @Local(argsOnly = true, name = "resourceManager") ResourceManager resourceManager) { if (resourceManager instanceof FabricMultiPackResourceManager flrm) { return ResourceLoaderImpl.sort(flrm.fabric$getPackType(), reloaders); } @@ -66,7 +66,8 @@ private static List sortProfiled(List Map, List>> loadTagsForRegistry(ResourceManager manager, ResourceKey> registryKey, TagLoader.ElementLookup> lookup, Operation, List>>> original, @Local(argsOnly = true) WritableRegistry registry) { + private static Map, List>> loadTagsForRegistry(ResourceManager manager, ResourceKey> registryKey, TagLoader.ElementLookup> lookup, Operation, List>>> original, @Local(argsOnly = true, name = "registry") WritableRegistry registry) { Map, List>> tags = original.call(manager, registryKey, lookup); registry.bindTags(tags); return tags; diff --git a/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/BucketItemMixin.java b/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/BucketItemMixin.java index fc5f065772b..ad26f31892b 100644 --- a/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/BucketItemMixin.java +++ b/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/BucketItemMixin.java @@ -43,7 +43,7 @@ public class BucketItemMixin { @ModifyVariable( method = "playEmptySound", at = @At("STORE"), - index = 4 + name = "soundEvent" ) private SoundEvent hookEmptyingSound(SoundEvent previous) { return FluidVariantAttributes.getHandlerOrDefault(content).getEmptySound(FluidVariant.of(content)).orElse(previous); diff --git a/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/CrafterBlockMixin.java b/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/CrafterBlockMixin.java index 1db6032aae5..d48db8489de 100644 --- a/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/CrafterBlockMixin.java +++ b/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/CrafterBlockMixin.java @@ -42,7 +42,7 @@ public class CrafterBlockMixin { // Inject after vanilla's attempts to insert the stack into an inventory. @Inject(method = "dispenseItem", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/item/ItemStack;isEmpty()Z")) - private void transferOrSpawnStack(ServerLevel level, BlockPos pos, CrafterBlockEntity blockEntity, ItemStack inputStack, BlockState state, RecipeHolder recipe, CallbackInfo ci, @Local Direction direction, @Local Container inventory, @Local(ordinal = 1) ItemStack itemStack) { + private void transferOrSpawnStack(ServerLevel level, BlockPos pos, CrafterBlockEntity blockEntity, ItemStack inputStack, BlockState state, RecipeHolder recipe, CallbackInfo ci, @Local(name = "direction") Direction direction, @Local(name = "into") Container inventory, @Local(name = "remaining") ItemStack itemStack) { if (inventory != null) { // Vanilla already found and tested an inventory, nothing else to do even if it failed to insert. return; diff --git a/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/HopperBlockEntityMixin.java b/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/HopperBlockEntityMixin.java index 9bf046de84c..5defeac4e24 100644 --- a/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/HopperBlockEntityMixin.java +++ b/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/HopperBlockEntityMixin.java @@ -52,7 +52,7 @@ public class HopperBlockEntityMixin { method = "ejectItems", cancellable = true ) - private static void hookInsert(Level level, BlockPos pos, HopperBlockEntity blockEntity, CallbackInfoReturnable cir, @Local Container targetInventory) { + private static void hookInsert(Level level, BlockPos pos, HopperBlockEntity blockEntity, CallbackInfoReturnable cir, @Local(name = "container") Container targetInventory) { // Let vanilla handle the transfer if it found an inventory. if (targetInventory != null) return; @@ -81,7 +81,7 @@ private static void hookInsert(Level level, BlockPos pos, HopperBlockEntity bloc method = "suckInItems(Lnet/minecraft/world/level/Level;Lnet/minecraft/world/level/block/entity/Hopper;)Z", cancellable = true ) - private static void hookExtract(Level level, Hopper hopper, CallbackInfoReturnable cir, @Local Container inputInventory) { + private static void hookExtract(Level level, Hopper hopper, CallbackInfoReturnable cir, @Local(name = "container") Container inputInventory) { // Let vanilla handle the transfer if it found an inventory. if (inputInventory != null) return; From 41ba36883aadaeb37d7d9b99db4a13332d3bb4dd Mon Sep 17 00:00:00 2001 From: modmuss50 Date: Sat, 24 Jan 2026 14:00:21 +0000 Subject: [PATCH 2/6] Add checkstyle rule + more fixes --- checkstyle.xml | 14 ++++++++++++++ .../mixin/item/BrewingStandBlockEntityMixin.java | 2 +- .../SimpleJsonResourceReloadListenerMixin.java | 2 +- .../indigo/renderer/SectionCompilerMixin.java | 2 +- 4 files changed, 17 insertions(+), 3 deletions(-) diff --git a/checkstyle.xml b/checkstyle.xml index 2574b46b00c..acaa08df136 100644 --- a/checkstyle.xml +++ b/checkstyle.xml @@ -284,6 +284,20 @@ + + + + + + diff --git a/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/BrewingStandBlockEntityMixin.java b/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/BrewingStandBlockEntityMixin.java index 6b6b7856dda..d269cadc56f 100644 --- a/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/BrewingStandBlockEntityMixin.java +++ b/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/BrewingStandBlockEntityMixin.java @@ -29,7 +29,7 @@ @Mixin(BrewingStandBlockEntity.class) public class BrewingStandBlockEntityMixin { @Redirect(method = "doBrew", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/item/Item;getCraftingRemainder()Lnet/minecraft/world/item/ItemStackTemplate;")) - private static ItemStackTemplate getCraftingRemainder(Item item, @Local(ordinal = 0) ItemStack stack) { + private static ItemStackTemplate getCraftingRemainder(Item item, @Local(name = "ingredient") ItemStack stack) { return stack.getCraftingRemainder(); } } diff --git a/fabric-loot-api-v3/src/main/java/net/fabricmc/fabric/mixin/loot/SimpleJsonResourceReloadListenerMixin.java b/fabric-loot-api-v3/src/main/java/net/fabricmc/fabric/mixin/loot/SimpleJsonResourceReloadListenerMixin.java index 581d6f9c541..660dad8c2e3 100644 --- a/fabric-loot-api-v3/src/main/java/net/fabricmc/fabric/mixin/loot/SimpleJsonResourceReloadListenerMixin.java +++ b/fabric-loot-api-v3/src/main/java/net/fabricmc/fabric/mixin/loot/SimpleJsonResourceReloadListenerMixin.java @@ -39,7 +39,7 @@ @Mixin(SimpleJsonResourceReloadListener.class) public class SimpleJsonResourceReloadListenerMixin { @Inject(method = "scanDirectory(Lnet/minecraft/server/packs/resources/ResourceManager;Lnet/minecraft/resources/FileToIdConverter;Lcom/mojang/serialization/DynamicOps;Lcom/mojang/serialization/Codec;Ljava/util/Map;)V", at = @At(value = "INVOKE_ASSIGN", target = "Lnet/minecraft/resources/FileToIdConverter;fileToId(Lnet/minecraft/resources/Identifier;)Lnet/minecraft/resources/Identifier;")) - private static void fillSourceMap(ResourceManager manager, FileToIdConverter fileToIdConverter, DynamicOps ops, Codec codec, Map result, CallbackInfo ci, @Local Map.Entry entry, @Local(ordinal = 1) Identifier id) { + private static void fillSourceMap(ResourceManager manager, FileToIdConverter fileToIdConverter, DynamicOps ops, Codec codec, Map result, CallbackInfo ci, @Local(name = "entry") Map.Entry entry, @Local(name = "id") Identifier id) { final String dirName = ((FileToIdConverterAccessor) fileToIdConverter).getPrefix(); if (!LootDataType.TABLE.registryKey().identifier().getPath().equals(dirName)) return; diff --git a/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/SectionCompilerMixin.java b/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/SectionCompilerMixin.java index 01c0686fc88..51c58b065aa 100644 --- a/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/SectionCompilerMixin.java +++ b/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/SectionCompilerMixin.java @@ -108,7 +108,7 @@ private void hookBuild(SectionPos sectionPos, RenderSectionRegion region, Vertex * which was specifically created to provide for enhanced terrain rendering. */ @Redirect(method = "compile", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/state/BlockState;getRenderShape()Lnet/minecraft/world/level/block/RenderShape;")) - private RenderShape hookBuildRenderBlock(BlockState blockState, SectionPos sectionPos, RenderSectionRegion renderRegion, VertexSorting vertexSorter, SectionBufferBuilderPack buffers, @Local(ordinal = 2) BlockPos blockPos) { + private RenderShape hookBuildRenderBlock(BlockState blockState, SectionPos sectionPos, RenderSectionRegion renderRegion, VertexSorting vertexSorter, SectionBufferBuilderPack buffers, @Local(name = "pos") BlockPos blockPos) { RenderShape renderShape = blockState.getRenderShape(); if (renderShape == RenderShape.MODEL) { From 0cdc1800206c479437582e21a46fb4616c66db69 Mon Sep 17 00:00:00 2001 From: modmuss50 Date: Sat, 24 Jan 2026 14:43:59 +0000 Subject: [PATCH 3/6] More fixes --- .../mixin/content/registry/PathfindingContextMixin.java | 2 +- .../fabric/mixin/entity/event/effect/LivingEntityMixin.java | 2 +- .../interaction/ServerGamePacketListenerImplMixin.java | 4 ++-- .../fabricmc/fabric/mixin/gametest/server/MainMixin.java | 2 +- .../mixin/event/lifecycle/client/ClientChunkCacheMixin.java | 6 +++--- .../fabric/mixin/client/model/loading/ModelBakeryMixin.java | 4 ++-- .../mixin/registry/sync/RegistryPatchGeneratorMixin.java | 3 ++- .../block/render/MushroomCowMushroomLayerMixin.java | 2 +- .../renderer/block/render/SnowGolemHeadLayerMixin.java | 2 +- .../client/indigo/renderer/BlockRenderDispatcherMixin.java | 2 +- .../client/rendering/fluid/LiquidBlockRendererMixin.java | 3 +-- .../mixin/resource/client/CreateWorldScreenMixin.java | 3 ++- .../mixin/resource/client/GameOptionsWriteVisitorMixin.java | 2 +- .../mixin/resource/DedicatedServerPropertiesMixin.java | 3 ++- 14 files changed, 21 insertions(+), 19 deletions(-) diff --git a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/PathfindingContextMixin.java b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/PathfindingContextMixin.java index 0f8d84dabf7..6d197c114b0 100644 --- a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/PathfindingContextMixin.java +++ b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/PathfindingContextMixin.java @@ -43,7 +43,7 @@ public abstract class PathfindingContextMixin { * Overrides the node type for the specified position, if the position is found as neighbor block in a path. */ @Inject(method = "getPathTypeFromState", at = @At(value = "INVOKE_ASSIGN", target = "Lnet/minecraft/core/BlockPos$MutableBlockPos;set(III)Lnet/minecraft/core/BlockPos$MutableBlockPos;"), cancellable = true) - private void onGetNodeType(int x, int y, int z, CallbackInfoReturnable cir, @Local BlockPos pos) { + private void onGetNodeType(int x, int y, int z, CallbackInfoReturnable cir, @Local(name = "pos") BlockPos pos) { final PathType neighborPathType = LandPathTypeRegistry.getPathType(getBlockState(pos), level(), pos, true); if (neighborPathType != null) { diff --git a/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/effect/LivingEntityMixin.java b/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/effect/LivingEntityMixin.java index 449be03c17d..57a0694e25e 100644 --- a/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/effect/LivingEntityMixin.java +++ b/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/effect/LivingEntityMixin.java @@ -179,7 +179,7 @@ private void beforeRemoveEffect(Holder holder, CallbackInfoReturnable target = "Ljava/util/Iterator;remove()V" ) ) - private void beforeExpireRemoveEffect(CallbackInfo ci, @Local MobEffectInstance effectInstance) { + private void beforeExpireRemoveEffect(CallbackInfo ci, @Local(name = "effect") MobEffectInstance effectInstance) { if (this.isClient()) { return; } diff --git a/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerGamePacketListenerImplMixin.java b/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerGamePacketListenerImplMixin.java index 85b7fc6baa0..45a1d8c36df 100644 --- a/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerGamePacketListenerImplMixin.java +++ b/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerGamePacketListenerImplMixin.java @@ -80,8 +80,8 @@ public ItemStack onPickItemFromEntity(Entity entity, Operation operat return ItemStack.EMPTY; } - @Inject(method = "handleInteract", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/level/ServerPlayer;getItemInHand(Lnet/minecraft/world/InteractionHand;)Lnet/minecraft/world/item/ItemStack;")) - public void handleInteract(ServerboundInteractPacket packet, CallbackInfo info, @Local Entity target) { + @Inject(method = "handleInteract", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/level/ServerPlayer;getItemInHand(Lnet/minecraft/world/InteractionHand;)Lnet/minecraft/world/item/ItemStack;"), cancellable = true) + public void handleInteract(ServerboundInteractPacket packet, CallbackInfo info, @Local(name = "target") Entity target) { Level level = player.level(); EntityHitResult hitResult = new EntityHitResult(target, packet.location().add(target.getX(), target.getY(), target.getZ())); diff --git a/fabric-gametest-api-v1/src/main/java/net/fabricmc/fabric/mixin/gametest/server/MainMixin.java b/fabric-gametest-api-v1/src/main/java/net/fabricmc/fabric/mixin/gametest/server/MainMixin.java index e95da801d8b..a2582a3694b 100644 --- a/fabric-gametest-api-v1/src/main/java/net/fabricmc/fabric/mixin/gametest/server/MainMixin.java +++ b/fabric-gametest-api-v1/src/main/java/net/fabricmc/fabric/mixin/gametest/server/MainMixin.java @@ -38,7 +38,7 @@ private static boolean isEulaAgreedTo(boolean isEulaAgreedTo) { // Inject after packRepository is stored @Inject(method = "main", cancellable = true, at = @At(value = "INVOKE_ASSIGN", target = "Lnet/minecraft/server/packs/repository/ServerPacksSource;createPackRepository(Lnet/minecraft/world/level/storage/LevelStorageSource$LevelStorageAccess;)Lnet/minecraft/server/packs/repository/PackRepository;")) - private static void main(String[] args, CallbackInfo info, @Local LevelStorageSource.LevelStorageAccess storageAccess, @Local PackRepository packRepository) { + private static void main(String[] args, CallbackInfo info, @Local(name = "access") LevelStorageSource.LevelStorageAccess storageAccess, @Local(name = "packRepository") PackRepository packRepository) { if (FabricGameTestRunner.ENABLED) { FabricGameTestRunner.runHeadlessServer(storageAccess, packRepository); info.cancel(); // Do not progress in starting the normal dedicated server diff --git a/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/ClientChunkCacheMixin.java b/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/ClientChunkCacheMixin.java index 500c52ad062..c642632b33e 100644 --- a/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/ClientChunkCacheMixin.java +++ b/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/ClientChunkCacheMixin.java @@ -50,14 +50,14 @@ private void onChunkLoad(int x, int z, FriendlyByteBuf friendlyByteBuf, Map highmap, Consumer consumer, CallbackInfoReturnable info, @Local LevelChunk levelChunk) { + private void onChunkUnload(int x, int z, FriendlyByteBuf buf, Map highmap, Consumer consumer, CallbackInfoReturnable info, @Local(name = "chunk") LevelChunk levelChunk) { if (levelChunk != null) { ClientChunkEvents.CHUNK_UNLOAD.invoker().onChunkUnload(this.level, levelChunk); } } @Inject(method = "drop", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/multiplayer/ClientChunkCache$Storage;drop(ILnet/minecraft/world/level/chunk/LevelChunk;)V")) - private void onChunkUnload(ChunkPos pos, CallbackInfo ci, @Local LevelChunk chunk) { + private void onChunkUnload(ChunkPos pos, CallbackInfo ci, @Local(name = "currentChunk") LevelChunk chunk) { ClientChunkEvents.CHUNK_UNLOAD.invoker().onChunkUnload(this.level, chunk); } @@ -68,7 +68,7 @@ private void onChunkUnload(ChunkPos pos, CallbackInfo ci, @Local LevelChunk chun target = "Lnet/minecraft/client/multiplayer/ClientChunkCache$Storage;inRange(II)Z" ) ) - private void onUpdateLoadDistance(int loadDistance, CallbackInfo ci, @Local ClientChunkCache.Storage clientChunkCacheStorage, @Local LevelChunk oldChunk, @Local ChunkPos chunkPos) { + private void onUpdateLoadDistance(int loadDistance, CallbackInfo ci, @Local(name = "newStorage") ClientChunkCache.Storage clientChunkCacheStorage, @Local(name = "chunk") LevelChunk oldChunk, @Local(name = "pos") ChunkPos chunkPos) { if (!clientChunkCacheStorage.inRange(chunkPos.x(), chunkPos.z())) { ClientChunkEvents.CHUNK_UNLOAD.invoker().onChunkUnload(this.level, oldChunk); } diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/ModelBakeryMixin.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/ModelBakeryMixin.java index 4e9b296ec29..7f37a6c2da1 100644 --- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/ModelBakeryMixin.java +++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/ModelBakeryMixin.java @@ -83,7 +83,7 @@ private BiFunction hoo } @ModifyReturnValue(method = "bakeModels", at = @At("RETURN")) - private CompletableFuture withExtraModels(CompletableFuture models, @Local Executor executor, @Local ModelBakery.ModelBakerImpl baker) { + private CompletableFuture withExtraModels(CompletableFuture models, @Local(argsOnly = true) Executor executor, @Local(name = "baker") ModelBakery.ModelBakerImpl baker) { if (fabric_eventDispatcher == null) return models; CompletableFuture, Object>> extraModels = ParallelMapTransform.schedule(fabric_eventDispatcher.getExtraModels(), (key, model) -> { @@ -112,7 +112,7 @@ private static BlockStateModel wrapBlockModelBake(BlockStateModel.UnbakedRoot un } @WrapOperation(method = "lambda$bakeModels$1", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/renderer/item/ItemModel$Unbaked;bake(Lnet/minecraft/client/renderer/item/ItemModel$BakingContext;)Lnet/minecraft/client/renderer/item/ItemModel;")) - private ItemModel wrapItemModelBake(ItemModel.Unbaked unbakedModel, ItemModel.BakingContext bakeContext, Operation operation, @Local Identifier itemId) { + private ItemModel wrapItemModelBake(ItemModel.Unbaked unbakedModel, ItemModel.BakingContext bakeContext, Operation operation, @Local(argsOnly = true) Identifier itemId) { if (fabric_eventDispatcher == null) { return operation.call(unbakedModel, bakeContext); } diff --git a/fabric-registry-sync-v0/src/main/java/net/fabricmc/fabric/mixin/registry/sync/RegistryPatchGeneratorMixin.java b/fabric-registry-sync-v0/src/main/java/net/fabricmc/fabric/mixin/registry/sync/RegistryPatchGeneratorMixin.java index 2c7453194a7..b0cbe546bbd 100644 --- a/fabric-registry-sync-v0/src/main/java/net/fabricmc/fabric/mixin/registry/sync/RegistryPatchGeneratorMixin.java +++ b/fabric-registry-sync-v0/src/main/java/net/fabricmc/fabric/mixin/registry/sync/RegistryPatchGeneratorMixin.java @@ -18,6 +18,7 @@ import java.util.List; +import org.objectweb.asm.Opcodes; import org.spongepowered.asm.mixin.Mixin; import org.spongepowered.asm.mixin.injection.At; import org.spongepowered.asm.mixin.injection.Redirect; @@ -29,7 +30,7 @@ @Mixin(RegistryPatchGenerator.class) class RegistryPatchGeneratorMixin { - @Redirect(at = @At(value = "FIELD", target = "Lnet/minecraft/resources/RegistryDataLoader;WORLDGEN_REGISTRIES:Ljava/util/List;"), method = "lambda$createLookup$0") + @Redirect(at = @At(value = "FIELD", target = "Lnet/minecraft/resources/RegistryDataLoader;WORLDGEN_REGISTRIES:Ljava/util/List;", opcode = Opcodes.GETSTATIC), method = "lambda$createLookup$0") private static List> getDynamicRegistries() { // Register cloners for all dynamic registries. return DynamicRegistries.getDynamicRegistries(); diff --git a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/MushroomCowMushroomLayerMixin.java b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/MushroomCowMushroomLayerMixin.java index 964ab273ae9..ec980279f29 100644 --- a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/MushroomCowMushroomLayerMixin.java +++ b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/MushroomCowMushroomLayerMixin.java @@ -34,7 +34,7 @@ abstract class MushroomCowMushroomLayerMixin { // Fix tinted quads being rendered completely black and provide the BlockState as context. @Redirect(method = "submitMushroomBlock", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/renderer/SubmitNodeCollector;submitBlockModel(Lcom/mojang/blaze3d/vertex/PoseStack;Lnet/minecraft/client/renderer/rendertype/RenderType;Lnet/minecraft/client/renderer/block/model/BlockStateModel;FFFIII)V")) - private void renderProxy(SubmitNodeCollector submitNodeCollector, PoseStack poseStack, RenderType renderType, BlockStateModel model, float r, float g, float b, int light, int overlay, int outlineColor, @Local BlockState blockState) { + private void renderProxy(SubmitNodeCollector submitNodeCollector, PoseStack poseStack, RenderType renderType, BlockStateModel model, float r, float g, float b, int light, int overlay, int outlineColor, @Local(argsOnly = true) BlockState blockState) { submitNodeCollector.submitBlockStateModel(poseStack, _ -> renderType, model, 1, 1, 1, light, overlay, outlineColor, EmptyBlockAndTintGetter.INSTANCE, BlockPos.ZERO, blockState); } } diff --git a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/SnowGolemHeadLayerMixin.java b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/SnowGolemHeadLayerMixin.java index 62998be3db6..52ff3dbd4f9 100644 --- a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/SnowGolemHeadLayerMixin.java +++ b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/SnowGolemHeadLayerMixin.java @@ -36,7 +36,7 @@ @Mixin(SnowGolemHeadLayer.class) abstract class SnowGolemHeadLayerMixin { @Redirect(method = "submit(Lcom/mojang/blaze3d/vertex/PoseStack;Lnet/minecraft/client/renderer/SubmitNodeCollector;ILnet/minecraft/client/renderer/entity/state/SnowGolemRenderState;FF)V", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/renderer/SubmitNodeCollector;submitBlockModel(Lcom/mojang/blaze3d/vertex/PoseStack;Lnet/minecraft/client/renderer/rendertype/RenderType;Lnet/minecraft/client/renderer/block/model/BlockStateModel;FFFIII)V")) - private void renderProxy(SubmitNodeCollector submitNodeCollector, PoseStack poseStack, RenderType renderType, BlockStateModel model, float r, float g, float b, int light, int overlay, int outlineColor, @Local SnowGolemRenderState renderState, @Local BlockState blockState) { + private void renderProxy(SubmitNodeCollector submitNodeCollector, PoseStack poseStack, RenderType renderType, BlockStateModel model, float r, float g, float b, int light, int overlay, int outlineColor, @Local(argsOnly = true) SnowGolemRenderState renderState, @Local(name = "pumpkinBlockState") BlockState blockState) { // If true, the chunk layer is an outline chunk layer, and we want all geometry to use this chunk layer. if (renderState.appearsGlowing() && renderState.isInvisible) { // Fix tinted quads being rendered completely black and provide the BlockState as context. diff --git a/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/BlockRenderDispatcherMixin.java b/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/BlockRenderDispatcherMixin.java index a9708041171..e468eba800a 100644 --- a/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/BlockRenderDispatcherMixin.java +++ b/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/BlockRenderDispatcherMixin.java @@ -47,7 +47,7 @@ abstract class BlockRenderDispatcherMixin { private ModelBlockRenderer modelRenderer; @Inject(method = "renderBreakingTexture(Lnet/minecraft/world/level/block/state/BlockState;Lnet/minecraft/core/BlockPos;Lnet/minecraft/world/level/BlockAndTintGetter;Lcom/mojang/blaze3d/vertex/PoseStack;Lcom/mojang/blaze3d/vertex/VertexConsumer;)V", at = @At(value = "INVOKE_ASSIGN", target = "Lnet/minecraft/client/renderer/block/BlockModelShaper;getBlockModel(Lnet/minecraft/world/level/block/state/BlockState;)Lnet/minecraft/client/renderer/block/model/BlockStateModel;"), cancellable = true) - private void afterGetModel(BlockState blockState, BlockPos blockPos, BlockAndTintGetter level, PoseStack poseStack, VertexConsumer vertexConsumer, CallbackInfo ci, @Local BlockStateModel model) { + private void afterGetModel(BlockState blockState, BlockPos blockPos, BlockAndTintGetter level, PoseStack poseStack, VertexConsumer vertexConsumer, CallbackInfo ci, @Local(name = "model") BlockStateModel model) { modelRenderer.render(level, model, blockState, blockPos, poseStack, layer -> vertexConsumer, true, blockState.getSeed(blockPos), OverlayTexture.NO_OVERLAY); ci.cancel(); diff --git a/fabric-rendering-fluids-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/fluid/LiquidBlockRendererMixin.java b/fabric-rendering-fluids-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/fluid/LiquidBlockRendererMixin.java index 097df953ca2..e43c6e06006 100644 --- a/fabric-rendering-fluids-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/fluid/LiquidBlockRendererMixin.java +++ b/fabric-rendering-fluids-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/fluid/LiquidBlockRendererMixin.java @@ -142,8 +142,7 @@ public int modTintColor(int original, BlockAndTintGetter level, BlockPos pos, Ve method = "tesselate", at = @At(value = "MIXINEXTRAS:EXPRESSION", ordinal = 0), slice = @Slice(from = @At(value = "FIELD", target = "Lnet/minecraft/client/renderer/block/LiquidBlockRenderer;waterOverlay:Lnet/minecraft/client/renderer/texture/TextureAtlasSprite;", opcode = Opcodes.GETFIELD)), - ordinal = 2 - ) + name = "sprite") private TextureAtlasSprite modifyOverlaySprite( TextureAtlasSprite waterOverlay, BlockAndTintGetter level, diff --git a/fabric-resource-loader-v1/src/client/java/net/fabricmc/fabric/mixin/resource/client/CreateWorldScreenMixin.java b/fabric-resource-loader-v1/src/client/java/net/fabricmc/fabric/mixin/resource/client/CreateWorldScreenMixin.java index 1e2d817a1d9..db3b977df96 100644 --- a/fabric-resource-loader-v1/src/client/java/net/fabricmc/fabric/mixin/resource/client/CreateWorldScreenMixin.java +++ b/fabric-resource-loader-v1/src/client/java/net/fabricmc/fabric/mixin/resource/client/CreateWorldScreenMixin.java @@ -19,6 +19,7 @@ import java.io.File; import com.mojang.datafixers.util.Pair; +import org.objectweb.asm.Opcodes; import org.spongepowered.asm.mixin.Mixin; import org.spongepowered.asm.mixin.Shadow; import org.spongepowered.asm.mixin.injection.At; @@ -55,7 +56,7 @@ private static PackRepository onCreateResManagerInit(PackRepository manager) { } @Redirect(method = "openCreateWorldScreen(Lnet/minecraft/client/Minecraft;Ljava/lang/Runnable;Ljava/util/function/Function;Lnet/minecraft/client/gui/screens/worldselection/WorldCreationContextMapper;Lnet/minecraft/resources/ResourceKey;Lnet/minecraft/client/gui/screens/worldselection/CreateWorldCallback;)V", - at = @At(value = "FIELD", target = "Lnet/minecraft/world/level/WorldDataConfiguration;DEFAULT:Lnet/minecraft/world/level/WorldDataConfiguration;", ordinal = 0)) + at = @At(value = "FIELD", target = "Lnet/minecraft/world/level/WorldDataConfiguration;DEFAULT:Lnet/minecraft/world/level/WorldDataConfiguration;", ordinal = 0, opcode = Opcodes.GETSTATIC)) private static WorldDataConfiguration replaceDefaultSettings() { return ModPackResourcesUtil.createDefaultDataConfiguration(); } diff --git a/fabric-resource-loader-v1/src/client/java/net/fabricmc/fabric/mixin/resource/client/GameOptionsWriteVisitorMixin.java b/fabric-resource-loader-v1/src/client/java/net/fabricmc/fabric/mixin/resource/client/GameOptionsWriteVisitorMixin.java index 9f45d427342..17f2267fd2b 100644 --- a/fabric-resource-loader-v1/src/client/java/net/fabricmc/fabric/mixin/resource/client/GameOptionsWriteVisitorMixin.java +++ b/fabric-resource-loader-v1/src/client/java/net/fabricmc/fabric/mixin/resource/client/GameOptionsWriteVisitorMixin.java @@ -53,7 +53,7 @@ private static List toPackListString(List packs) { @SuppressWarnings("unchecked") @ModifyArg(method = "process(Ljava/lang/String;Ljava/lang/Object;Ljava/util/function/Function;Ljava/util/function/Function;)Ljava/lang/Object;", at = @At(value = "INVOKE", target = "Ljava/util/function/Function;apply(Ljava/lang/Object;)Ljava/lang/Object;")) - private T skipHiddenPacks(T value, @Local String key) { + private T skipHiddenPacks(T value, @Local(argsOnly = true) String key) { if ("resourcePacks".equals(key) && value instanceof List) { return (T) toPackListString((List) value); } diff --git a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/DedicatedServerPropertiesMixin.java b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/DedicatedServerPropertiesMixin.java index 0c8108f2fc1..eac94559754 100644 --- a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/DedicatedServerPropertiesMixin.java +++ b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/DedicatedServerPropertiesMixin.java @@ -16,6 +16,7 @@ package net.fabricmc.fabric.mixin.resource; +import org.objectweb.asm.Opcodes; import org.spongepowered.asm.mixin.Mixin; import org.spongepowered.asm.mixin.injection.At; import org.spongepowered.asm.mixin.injection.Redirect; @@ -27,7 +28,7 @@ @Mixin(DedicatedServerProperties.class) public class DedicatedServerPropertiesMixin { - @Redirect(method = "", at = @At(value = "FIELD", target = "Lnet/minecraft/world/level/WorldDataConfiguration;DEFAULT:Lnet/minecraft/world/level/WorldDataConfiguration;")) + @Redirect(method = "", at = @At(value = "FIELD", target = "Lnet/minecraft/world/level/WorldDataConfiguration;DEFAULT:Lnet/minecraft/world/level/WorldDataConfiguration;", opcode = Opcodes.GETSTATIC)) private WorldDataConfiguration replaceDefaultDataConfiguration() { return ModPackResourcesUtil.createDefaultDataConfiguration(); } From 35c45db0e41eb5d7c93df60774214fb1cbe17028 Mon Sep 17 00:00:00 2001 From: modmuss50 Date: Sat, 24 Jan 2026 16:39:23 +0000 Subject: [PATCH 4/6] Renames --- .../client/RenderRegionCacheMixin.java | 4 ++-- .../gametest/world/CreateWorldScreenMixin.java | 4 ++-- .../content/registry/WalkNodeEvaluatorMixin.java | 4 ++-- .../mixin/datagen/client/ModelProviderMixin.java | 10 +++++----- .../fabric/mixin/datagen/TagsProviderMixin.java | 4 ++-- .../mixin/entity/event/LivingEntityMixin.java | 4 ++-- .../entity/event/effect/LivingEntityMixin.java | 4 ++-- .../client/MultiPlayerGameModeMixin.java | 4 ++-- .../interaction/ServerPlayerGameModeMixin.java | 10 +++++----- .../fabric/mixin/item/AnvilMenuMixin.java | 4 ++-- .../mixin/item/BrewingStandBlockEntityMixin.java | 4 ++-- .../lifecycle/client/ClientChunkCacheMixin.java | 16 ++++++++-------- .../event/lifecycle/client/LevelChunkMixin.java | 16 ++++++++-------- .../mixin/event/lifecycle/LivingEntityMixin.java | 4 ++-- .../event/lifecycle/MinecraftServerMixin.java | 4 ++-- .../event/lifecycle/server/LevelChunkMixin.java | 10 +++++----- .../loot/ReloadableServerRegistriesMixin.java | 4 ++-- .../client/model/loading/ModelManagerMixin.java | 6 +++--- .../fabric/mixin/particle/EntityMixin.java | 4 ++-- .../fabric/mixin/recipe/sync/RecipeMapMixin.java | 6 +++--- .../particle/ScreenEffectRendererMixin.java | 4 ++-- .../block/render/ItemFrameRendererMixin.java | 4 ++-- .../block/render/SnowGolemHeadLayerMixin.java | 6 +++--- .../renderer/ItemStackRenderStateMixin.java | 4 ++-- .../indigo/renderer/SectionCompilerMixin.java | 12 ++++++------ .../fluid/LiquidBlockRendererMixin.java | 4 ++-- .../client/rendering/LayerDefinitionsMixin.java | 6 +++--- .../mixin/resource/conditions/PackMixin.java | 4 ++-- ...RegistryLoadTaskPendingRegistrationMixin.java | 4 ++-- .../mixin/resource/DataPackCommandMixin.java | 4 ++-- .../mixin/resource/PackRepositoryMixin.java | 16 ++++++++-------- .../resource/ReloadableServerResourcesMixin.java | 4 ++-- .../fabric/mixin/transfer/CrafterBlockMixin.java | 10 +++++----- .../mixin/transfer/HopperBlockEntityMixin.java | 8 ++++---- 34 files changed, 108 insertions(+), 108 deletions(-) diff --git a/fabric-block-getter-api-v2/src/client/java/net/fabricmc/fabric/mixin/blockgetter/client/RenderRegionCacheMixin.java b/fabric-block-getter-api-v2/src/client/java/net/fabricmc/fabric/mixin/blockgetter/client/RenderRegionCacheMixin.java index eb5697ad235..e55f4a8afee 100644 --- a/fabric-block-getter-api-v2/src/client/java/net/fabricmc/fabric/mixin/blockgetter/client/RenderRegionCacheMixin.java +++ b/fabric-block-getter-api-v2/src/client/java/net/fabricmc/fabric/mixin/blockgetter/client/RenderRegionCacheMixin.java @@ -50,7 +50,7 @@ public abstract class RenderRegionCacheMixin { private static final Logger LOGGER = LoggerFactory.getLogger(RenderRegionCacheMixin.class); @Inject(method = "createRegion", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/renderer/chunk/RenderRegionCache;getSectionDataCopy(Lnet/minecraft/world/level/Level;III)Lnet/minecraft/client/renderer/chunk/SectionCopy;")) - private void copyDataForChunk(Level level, long packedChunkPos, CallbackInfoReturnable cir, @Share("dataMap") LocalRef> mapRef, @Local(name = "regionSectionX") int x, @Local(name = "regionSectionY") int y, @Local(name = "regionSectionZ") int z) { + private void copyDataForChunk(Level level, long packedChunkPos, CallbackInfoReturnable cir, @Share("dataMap") LocalRef> mapRef, @Local(name = "regionSectionX") int regionSectionX, @Local(name = "regionSectionY") int regionSectionY, @Local(name = "regionSectionZ") int regionSectionZ) { // Hash maps in chunks should generally not be modified outside of client thread // but does happen in practice, due to mods or inconsistent vanilla behaviors, causing // CMEs when we iterate the map. (Vanilla does not iterate these maps when it builds @@ -59,7 +59,7 @@ private void copyDataForChunk(Level level, long packedChunkPos, CallbackInfoRetu // We handle this simply by retrying until it works. Ugly but effective. while (true) { try { - mapRef.set(mapChunk(level.getChunk(x, z), SectionPos.of(packedChunkPos), mapRef.get())); + mapRef.set(mapChunk(level.getChunk(regionSectionX, regionSectionZ), SectionPos.of(packedChunkPos), mapRef.get())); break; } catch (ConcurrentModificationException e) { final int count = ERROR_COUNTER.incrementAndGet(); diff --git a/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/world/CreateWorldScreenMixin.java b/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/world/CreateWorldScreenMixin.java index 0e050ee9dae..89e2bf819b7 100644 --- a/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/world/CreateWorldScreenMixin.java +++ b/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/world/CreateWorldScreenMixin.java @@ -38,9 +38,9 @@ @Mixin(CreateWorldScreen.class) public class CreateWorldScreenMixin { @Inject(method = "onCreate", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/gui/screens/worldselection/WorldOpenFlows;confirmWorldCreation(Lnet/minecraft/client/Minecraft;Lnet/minecraft/client/gui/screens/worldselection/CreateWorldScreen;Lcom/mojang/serialization/Lifecycle;Ljava/lang/Runnable;Z)V"), cancellable = true) - private void createLevelDataForServers(CallbackInfo ci, @Local(name = "finalLayers") LayeredRegistryAccess registries, @Local(name = "worldData") PrimaryLevelData levelData) { + private void createLevelDataForServers(CallbackInfo ci, @Local(name = "finalLayers") LayeredRegistryAccess finalLayers, @Local(name = "worldData") PrimaryLevelData worldData) { if (DedicatedServerImplUtil.saveLevelDataTo != null) { - CompoundTag levelDatInner = levelData.createTag(registries.compositeAccess(), null); + CompoundTag levelDatInner = worldData.createTag(finalLayers.compositeAccess(), null); CompoundTag levelDat = new CompoundTag(); levelDat.put("Data", levelDatInner); diff --git a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/WalkNodeEvaluatorMixin.java b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/WalkNodeEvaluatorMixin.java index d2a2ea79b55..22cffad9461 100644 --- a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/WalkNodeEvaluatorMixin.java +++ b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/WalkNodeEvaluatorMixin.java @@ -37,8 +37,8 @@ public class WalkNodeEvaluatorMixin { * Overrides the node type for the specified position, if the position is a direct target in a path. */ @Inject(method = "getPathTypeFromState", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/state/BlockState;getBlock()Lnet/minecraft/world/level/block/Block;"), cancellable = true) - private static void getCommonNodeType(BlockGetter level, BlockPos pos, CallbackInfoReturnable cir, @Local(name = "blockState") BlockState state) { - PathType pathType = LandPathTypeRegistry.getPathType(state, level, pos, false); + private static void getCommonNodeType(BlockGetter level, BlockPos pos, CallbackInfoReturnable cir, @Local(name = "blockState") BlockState blockState) { + PathType pathType = LandPathTypeRegistry.getPathType(blockState, level, pos, false); if (pathType != null) { cir.setReturnValue(pathType); diff --git a/fabric-data-generation-api-v1/src/client/java/net/fabricmc/fabric/mixin/datagen/client/ModelProviderMixin.java b/fabric-data-generation-api-v1/src/client/java/net/fabricmc/fabric/mixin/datagen/client/ModelProviderMixin.java index d2183c5966c..69597daa215 100644 --- a/fabric-data-generation-api-v1/src/client/java/net/fabricmc/fabric/mixin/datagen/client/ModelProviderMixin.java +++ b/fabric-data-generation-api-v1/src/client/java/net/fabricmc/fabric/mixin/datagen/client/ModelProviderMixin.java @@ -73,10 +73,10 @@ private void registerItemModels(ItemModelGenerators instance, Operation or @Inject(method = "run", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/data/models/BlockModelGenerators;run()V")) private void setFabricPackOutput(CachedOutput output, CallbackInfoReturnable> cir, - @Local(name = "blockStateGenerators") ModelProvider.BlockStateGeneratorCollector blockStateGeneratorCollector, - @Local(name = "itemModels") ModelProvider.ItemInfoCollector itemInfoCollectors) { - ((FabricModelProviderDefinitions) blockStateGeneratorCollector).setFabricPackOutput(fabricPackOutput); - ((FabricModelProviderDefinitions) itemInfoCollectors).setFabricPackOutput(fabricPackOutput); - ((FabricItemAssetDefinitions) itemInfoCollectors).fabric_setProcessedBlocks(blockStateGeneratorCollector.generators.keySet()); + @Local(name = "blockStateGenerators") ModelProvider.BlockStateGeneratorCollector blockStateGenerators, + @Local(name = "itemModels") ModelProvider.ItemInfoCollector itemModels) { + ((FabricModelProviderDefinitions) blockStateGenerators).setFabricPackOutput(fabricPackOutput); + ((FabricModelProviderDefinitions) itemModels).setFabricPackOutput(fabricPackOutput); + ((FabricItemAssetDefinitions) itemModels).fabric_setProcessedBlocks(blockStateGenerators.generators.keySet()); } } diff --git a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/TagsProviderMixin.java b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/TagsProviderMixin.java index 47e9ae0e7c8..ae1488d0764 100644 --- a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/TagsProviderMixin.java +++ b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/TagsProviderMixin.java @@ -58,8 +58,8 @@ private void initPathResolver(PackOutput output, ResourceKey(Ljava/util/List;Z)V"), index = 1) - private boolean addReplaced(boolean replaced, @Local(name = "builder") TagBuilder tagBuilder) { - if (tagBuilder instanceof FabricTagBuilder fabricTagBuilder) { + private boolean addReplaced(boolean replaced, @Local(name = "builder") TagBuilder builder) { + if (builder instanceof FabricTagBuilder fabricTagBuilder) { return fabricTagBuilder.fabric_isReplaced(); } diff --git a/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/LivingEntityMixin.java b/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/LivingEntityMixin.java index 242de8984b0..1e392c3824e 100644 --- a/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/LivingEntityMixin.java +++ b/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/LivingEntityMixin.java @@ -85,9 +85,9 @@ private void beforeDamage(ServerLevel level, DamageSource source, float amount, } @Inject(method = "hurtServer", at = @At("TAIL")) - private void afterDamage(ServerLevel level, DamageSource source, float amount, CallbackInfoReturnable cir, @Local(name = "originalDamage") float dealt, @Local(name = "blocked") boolean blocked) { + private void afterDamage(ServerLevel level, DamageSource source, float amount, CallbackInfoReturnable cir, @Local(name = "originalDamage") float originalDamage, @Local(name = "blocked") boolean blocked) { if (!isDeadOrDying()) { - ServerLivingEntityEvents.AFTER_DAMAGE.invoker().afterDamage((LivingEntity) (Object) this, source, dealt, amount, blocked); + ServerLivingEntityEvents.AFTER_DAMAGE.invoker().afterDamage((LivingEntity) (Object) this, source, originalDamage, amount, blocked); } } diff --git a/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/effect/LivingEntityMixin.java b/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/effect/LivingEntityMixin.java index 57a0694e25e..51ec15682bb 100644 --- a/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/effect/LivingEntityMixin.java +++ b/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/effect/LivingEntityMixin.java @@ -179,13 +179,13 @@ private void beforeRemoveEffect(Holder holder, CallbackInfoReturnable target = "Ljava/util/Iterator;remove()V" ) ) - private void beforeExpireRemoveEffect(CallbackInfo ci, @Local(name = "effect") MobEffectInstance effectInstance) { + private void beforeExpireRemoveEffect(CallbackInfo ci, @Local(name = "effect") MobEffectInstance effect) { if (this.isClient()) { return; } ServerMobEffectEvents.BEFORE_REMOVE.invoker() - .beforeRemove(effectInstance, this.self(), MobEffectUtil.getCommandContext()); + .beforeRemove(effect, this.self(), MobEffectUtil.getCommandContext()); } @Inject( diff --git a/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MultiPlayerGameModeMixin.java b/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MultiPlayerGameModeMixin.java index f4cc1366467..2d03e488b6d 100644 --- a/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MultiPlayerGameModeMixin.java +++ b/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MultiPlayerGameModeMixin.java @@ -91,8 +91,8 @@ private void fabric_fireAttackBlockCallback(BlockPos pos, Direction direction, C } @Inject(method = "destroyBlock", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/Block;destroy(Lnet/minecraft/world/level/LevelAccessor;Lnet/minecraft/core/BlockPos;Lnet/minecraft/world/level/block/state/BlockState;)V")) - private void fabric$onBlockBroken(BlockPos pos, CallbackInfoReturnable cir, @Local(name = "oldState") BlockState blockState) { - ClientPlayerBlockBreakEvents.AFTER.invoker().afterBlockBreak(minecraft.level, minecraft.player, pos, blockState); + private void fabric$onBlockBroken(BlockPos pos, CallbackInfoReturnable cir, @Local(name = "oldState") BlockState oldState) { + ClientPlayerBlockBreakEvents.AFTER.invoker().afterBlockBreak(minecraft.level, minecraft.player, pos, oldState); } @Inject(at = @At(value = "INVOKE", target = "Lnet/minecraft/client/multiplayer/MultiPlayerGameMode;startPrediction(Lnet/minecraft/client/multiplayer/ClientLevel;Lnet/minecraft/client/multiplayer/prediction/PredictiveAction;)V"), method = "useItemOn", cancellable = true) diff --git a/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerPlayerGameModeMixin.java b/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerPlayerGameModeMixin.java index d74eb4251ea..37580d2c10d 100644 --- a/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerPlayerGameModeMixin.java +++ b/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerPlayerGameModeMixin.java @@ -104,18 +104,18 @@ public void interactItem(ServerPlayer player, Level level, ItemStack stack, Inte } @Inject(at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/Block;playerWillDestroy(Lnet/minecraft/world/level/Level;Lnet/minecraft/core/BlockPos;Lnet/minecraft/world/level/block/state/BlockState;Lnet/minecraft/world/entity/player/Player;)Lnet/minecraft/world/level/block/state/BlockState;"), method = "destroyBlock", cancellable = true) - private void breakBlock(BlockPos pos, CallbackInfoReturnable cir, @Local(name = "blockEntity") BlockEntity entity, @Local(name = "state") BlockState state) { - boolean result = PlayerBlockBreakEvents.BEFORE.invoker().beforeBlockBreak(this.level, this.player, pos, state, entity); + private void breakBlock(BlockPos pos, CallbackInfoReturnable cir, @Local(name = "blockEntity") BlockEntity blockEntity, @Local(name = "state") BlockState state) { + boolean result = PlayerBlockBreakEvents.BEFORE.invoker().beforeBlockBreak(this.level, this.player, pos, state, blockEntity); if (!result) { - PlayerBlockBreakEvents.CANCELED.invoker().onBlockBreakCanceled(this.level, this.player, pos, state, entity); + PlayerBlockBreakEvents.CANCELED.invoker().onBlockBreakCanceled(this.level, this.player, pos, state, blockEntity); cir.setReturnValue(false); } } @Inject(at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/Block;destroy(Lnet/minecraft/world/level/LevelAccessor;Lnet/minecraft/core/BlockPos;Lnet/minecraft/world/level/block/state/BlockState;)V"), method = "destroyBlock") - private void onBlockBroken(BlockPos pos, CallbackInfoReturnable cir, @Local(name = "blockEntity") BlockEntity entity, @Local(name = "adjustedState") BlockState state) { - PlayerBlockBreakEvents.AFTER.invoker().afterBlockBreak(this.level, this.player, pos, state, entity); + private void onBlockBroken(BlockPos pos, CallbackInfoReturnable cir, @Local(name = "blockEntity") BlockEntity blockEntity, @Local(name = "adjustedState") BlockState adjustedState) { + PlayerBlockBreakEvents.AFTER.invoker().afterBlockBreak(this.level, this.player, pos, adjustedState, blockEntity); } } diff --git a/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/AnvilMenuMixin.java b/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/AnvilMenuMixin.java index c34200de53f..a178fdf89b7 100644 --- a/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/AnvilMenuMixin.java +++ b/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/AnvilMenuMixin.java @@ -47,7 +47,7 @@ abstract class AnvilMenuMixin extends ItemCombinerMenu { target = "Lnet/minecraft/world/item/enchantment/Enchantment;canEnchant(Lnet/minecraft/world/item/ItemStack;)Z" ) ) - private boolean callAllowEnchantingEvent(Enchantment instance, ItemStack stack, @Local(name = "enchantmentHolder") Holder holder) { - return stack.canBeEnchantedWith(holder, EnchantingContext.ACCEPTABLE); + private boolean callAllowEnchantingEvent(Enchantment instance, ItemStack stack, @Local(name = "enchantmentHolder") Holder enchantmentHolder) { + return stack.canBeEnchantedWith(enchantmentHolder, EnchantingContext.ACCEPTABLE); } } diff --git a/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/BrewingStandBlockEntityMixin.java b/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/BrewingStandBlockEntityMixin.java index d269cadc56f..6859b309ab8 100644 --- a/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/BrewingStandBlockEntityMixin.java +++ b/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/BrewingStandBlockEntityMixin.java @@ -29,7 +29,7 @@ @Mixin(BrewingStandBlockEntity.class) public class BrewingStandBlockEntityMixin { @Redirect(method = "doBrew", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/item/Item;getCraftingRemainder()Lnet/minecraft/world/item/ItemStackTemplate;")) - private static ItemStackTemplate getCraftingRemainder(Item item, @Local(name = "ingredient") ItemStack stack) { - return stack.getCraftingRemainder(); + private static ItemStackTemplate getCraftingRemainder(Item item, @Local(name = "ingredient") ItemStack ingredient) { + return ingredient.getCraftingRemainder(); } } diff --git a/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/ClientChunkCacheMixin.java b/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/ClientChunkCacheMixin.java index c642632b33e..209dd9dbb86 100644 --- a/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/ClientChunkCacheMixin.java +++ b/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/ClientChunkCacheMixin.java @@ -50,15 +50,15 @@ private void onChunkLoad(int x, int z, FriendlyByteBuf friendlyByteBuf, Map highmap, Consumer consumer, CallbackInfoReturnable info, @Local(name = "chunk") LevelChunk levelChunk) { - if (levelChunk != null) { - ClientChunkEvents.CHUNK_UNLOAD.invoker().onChunkUnload(this.level, levelChunk); + private void onChunkUnload(int x, int z, FriendlyByteBuf buf, Map highmap, Consumer consumer, CallbackInfoReturnable info, @Local(name = "chunk") LevelChunk chunk) { + if (chunk != null) { + ClientChunkEvents.CHUNK_UNLOAD.invoker().onChunkUnload(this.level, chunk); } } @Inject(method = "drop", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/multiplayer/ClientChunkCache$Storage;drop(ILnet/minecraft/world/level/chunk/LevelChunk;)V")) - private void onChunkUnload(ChunkPos pos, CallbackInfo ci, @Local(name = "currentChunk") LevelChunk chunk) { - ClientChunkEvents.CHUNK_UNLOAD.invoker().onChunkUnload(this.level, chunk); + private void onChunkUnload(ChunkPos pos, CallbackInfo ci, @Local(name = "currentChunk") LevelChunk currentChunk) { + ClientChunkEvents.CHUNK_UNLOAD.invoker().onChunkUnload(this.level, currentChunk); } @Inject( @@ -68,9 +68,9 @@ private void onChunkUnload(ChunkPos pos, CallbackInfo ci, @Local(name = "current target = "Lnet/minecraft/client/multiplayer/ClientChunkCache$Storage;inRange(II)Z" ) ) - private void onUpdateLoadDistance(int loadDistance, CallbackInfo ci, @Local(name = "newStorage") ClientChunkCache.Storage clientChunkCacheStorage, @Local(name = "chunk") LevelChunk oldChunk, @Local(name = "pos") ChunkPos chunkPos) { - if (!clientChunkCacheStorage.inRange(chunkPos.x(), chunkPos.z())) { - ClientChunkEvents.CHUNK_UNLOAD.invoker().onChunkUnload(this.level, oldChunk); + private void onUpdateLoadDistance(int loadDistance, CallbackInfo ci, @Local(name = "newStorage") ClientChunkCache.Storage newStorage, @Local(name = "chunk") LevelChunk chunk, @Local(name = "pos") ChunkPos pos) { + if (!newStorage.inRange(pos.x(), pos.z())) { + ClientChunkEvents.CHUNK_UNLOAD.invoker().onChunkUnload(this.level, chunk); } } } diff --git a/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/LevelChunkMixin.java b/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/LevelChunkMixin.java index c33d8bb8d45..9ec1b54e80d 100644 --- a/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/LevelChunkMixin.java +++ b/fabric-lifecycle-events-v1/src/client/java/net/fabricmc/fabric/mixin/event/lifecycle/client/LevelChunkMixin.java @@ -62,12 +62,12 @@ private V onLoadBlockEntity(V removedBlockEntity, BlockEntity blockEntity) { } @Inject(method = "setBlockEntity", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/entity/BlockEntity;setRemoved()V", shift = At.Shift.AFTER)) - private void onRemoveBlockEntity(BlockEntity blockEntity, CallbackInfo info, @Local(name = "previousEntry") BlockEntity removedBlockEntity) { - if (removedBlockEntity != null) { + private void onRemoveBlockEntity(BlockEntity blockEntity, CallbackInfo info, @Local(name = "previousEntry") BlockEntity previousEntry) { + if (previousEntry != null) { if (this.getLevel() instanceof ServerLevel) { - ServerBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removedBlockEntity, (ServerLevel) this.getLevel()); + ServerBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(previousEntry, (ServerLevel) this.getLevel()); } else if (this.getLevel() instanceof ClientLevel) { - ClientBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removedBlockEntity, (ClientLevel) this.getLevel()); + ClientBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(previousEntry, (ClientLevel) this.getLevel()); } } } @@ -90,12 +90,12 @@ private Object onRemoveBlockEntity(Map map, K key) { } @Inject(method = "removeBlockEntity", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/entity/BlockEntity;setRemoved()V")) - private void onRemoveBlockEntity(BlockPos pos, CallbackInfo ci, @Local(name = "removeThis") @Nullable BlockEntity removed) { - if (removed != null) { + private void onRemoveBlockEntity(BlockPos pos, CallbackInfo ci, @Local(name = "removeThis") @Nullable BlockEntity removeThis) { + if (removeThis != null) { if (this.getLevel() instanceof ServerLevel) { - ServerBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removed, (ServerLevel) this.getLevel()); + ServerBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removeThis, (ServerLevel) this.getLevel()); } else if (this.getLevel() instanceof ClientLevel) { - ClientBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removed, (ClientLevel) this.getLevel()); + ClientBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removeThis, (ClientLevel) this.getLevel()); } } } diff --git a/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/LivingEntityMixin.java b/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/LivingEntityMixin.java index 64f9fdcb77a..eb8f6507d39 100644 --- a/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/LivingEntityMixin.java +++ b/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/LivingEntityMixin.java @@ -34,7 +34,7 @@ @Mixin(LivingEntity.class) public abstract class LivingEntityMixin { @Inject(method = "collectEquipmentChanges", at = @At(value = "INVOKE", target = "Ljava/util/Map;put(Ljava/lang/Object;Ljava/lang/Object;)Ljava/lang/Object;")) - private void getEquipmentChanges(CallbackInfoReturnable<@Nullable Map> cir, @Local(name = "slot") EquipmentSlot equipmentSlot, @Local(name = "previous") ItemStack previousStack, @Local(name = "current") ItemStack currentStack) { - ServerEntityEvents.EQUIPMENT_CHANGE.invoker().onChange((LivingEntity) (Object) this, equipmentSlot, previousStack, currentStack); + private void getEquipmentChanges(CallbackInfoReturnable<@Nullable Map> cir, @Local(name = "slot") EquipmentSlot slot, @Local(name = "previous") ItemStack previous, @Local(name = "current") ItemStack current) { + ServerEntityEvents.EQUIPMENT_CHANGE.invoker().onChange((LivingEntity) (Object) this, slot, previous, current); } } diff --git a/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/MinecraftServerMixin.java b/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/MinecraftServerMixin.java index 297f3983192..dcdf88734fc 100644 --- a/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/MinecraftServerMixin.java +++ b/fabric-lifecycle-events-v1/src/main/java/net/fabricmc/fabric/mixin/event/lifecycle/MinecraftServerMixin.java @@ -82,8 +82,8 @@ private V onLoadWorld(Map levels, K dimension, V level, Operation V onLoadBlockEntity(V removedBlockEntity, BlockEntity blockEntity) { } @Inject(method = "setBlockEntity", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/entity/BlockEntity;setRemoved()V", shift = At.Shift.AFTER)) - private void onRemoveBlockEntity(BlockEntity blockEntity, CallbackInfo info, @Local(name = "previousEntry") BlockEntity removedBlockEntity) { + private void onRemoveBlockEntity(BlockEntity blockEntity, CallbackInfo info, @Local(name = "previousEntry") BlockEntity previousEntry) { if (this.getLevel() instanceof ServerLevel) { - ServerBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removedBlockEntity, (ServerLevel) this.getLevel()); + ServerBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(previousEntry, (ServerLevel) this.getLevel()); } } @@ -83,9 +83,9 @@ private Object onRemoveBlockEntity(Map map, K key) { } @Inject(method = "removeBlockEntity", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/entity/BlockEntity;setRemoved()V")) - private void onRemoveBlockEntity(BlockPos pos, CallbackInfo ci, @Local(name = "removeThis") @Nullable BlockEntity removed) { - if (removed != null && this.getLevel() instanceof ServerLevel) { - ServerBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removed, (ServerLevel) this.getLevel()); + private void onRemoveBlockEntity(BlockPos pos, CallbackInfo ci, @Local(name = "removeThis") @Nullable BlockEntity removeThis) { + if (removeThis != null && this.getLevel() instanceof ServerLevel) { + ServerBlockEntityEvents.BLOCK_ENTITY_UNLOAD.invoker().onUnload(removeThis, (ServerLevel) this.getLevel()); } } } diff --git a/fabric-loot-api-v3/src/main/java/net/fabricmc/fabric/mixin/loot/ReloadableServerRegistriesMixin.java b/fabric-loot-api-v3/src/main/java/net/fabricmc/fabric/mixin/loot/ReloadableServerRegistriesMixin.java index cf3ed1d39fb..68b561b6765 100644 --- a/fabric-loot-api-v3/src/main/java/net/fabricmc/fabric/mixin/loot/ReloadableServerRegistriesMixin.java +++ b/fabric-loot-api-v3/src/main/java/net/fabricmc/fabric/mixin/loot/ReloadableServerRegistriesMixin.java @@ -82,8 +82,8 @@ private static CompletableFuture> removeOps } @Inject(method = "lambda$scheduleRegistryLoad$0", at = @At(value = "INVOKE", target = "Ljava/util/Map;forEach(Ljava/util/function/BiConsumer;)V")) - private static void modifyLootTable(LootDataType lootDataType, ResourceManager resourceManager, RegistryOps registryOps, CallbackInfoReturnable> cir, @Local(name = "elements") Map map) { - map.replaceAll((identifier, t) -> modifyLootTable(t, identifier, registryOps)); + private static void modifyLootTable(LootDataType lootDataType, ResourceManager resourceManager, RegistryOps registryOps, CallbackInfoReturnable> cir, @Local(name = "elements") Map elements) { + elements.replaceAll((identifier, t) -> modifyLootTable(t, identifier, registryOps)); } @Unique diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/ModelManagerMixin.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/ModelManagerMixin.java index a5cc8766a99..193a2bc7b5c 100644 --- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/ModelManagerMixin.java +++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/ModelManagerMixin.java @@ -118,12 +118,12 @@ private Function withModelDispatcher(Function function) { @Inject(method = "discoverModelDependencies", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/resources/model/ModelDiscovery;resolve()Ljava/util/Map;")) private static void resolveExtraModels( - Map modelMap, BlockStateModelLoader.LoadedModels stateDefinition, ClientItemInfoLoader.LoadedClientInfos result, CallbackInfoReturnable cir, - @Local(name = "result") ModelDiscovery collector + Map modelMap, BlockStateModelLoader.LoadedModels stateDefinition, ClientItemInfoLoader.LoadedClientInfos loadedClientInfos, CallbackInfoReturnable cir, + @Local(name = "result") ModelDiscovery result ) { // We know eventDispatcherFuture is available, as it is required by the item and block models (hookModels). ModelLoadingEventDispatcher eventDispatcher = ModelLoadingEventDispatcher.CURRENT.get(); - if (eventDispatcher != null) eventDispatcher.getExtraModels().values().forEach(collector::addRoot); + if (eventDispatcher != null) eventDispatcher.getExtraModels().values().forEach(result::addRoot); } @Inject(method = "apply", at = @At(value = "RETURN")) diff --git a/fabric-particles-v1/src/main/java/net/fabricmc/fabric/mixin/particle/EntityMixin.java b/fabric-particles-v1/src/main/java/net/fabricmc/fabric/mixin/particle/EntityMixin.java index b026366ebce..cd048937565 100644 --- a/fabric-particles-v1/src/main/java/net/fabricmc/fabric/mixin/particle/EntityMixin.java +++ b/fabric-particles-v1/src/main/java/net/fabricmc/fabric/mixin/particle/EntityMixin.java @@ -30,8 +30,8 @@ @Mixin(Entity.class) abstract class EntityMixin { @ModifyExpressionValue(method = "spawnSprintParticle", at = @At(value = "NEW", target = "(Lnet/minecraft/core/particles/ParticleType;Lnet/minecraft/world/level/block/state/BlockState;)Lnet/minecraft/core/particles/BlockParticleOption;")) - private BlockParticleOption modifyBlockStateParticleOption(BlockParticleOption original, @Local(name = "pos") BlockPos blockPos) { - ((BlockParticleOptionExtension) original).fabric_setBlockPos(blockPos); + private BlockParticleOption modifyBlockStateParticleOption(BlockParticleOption original, @Local(name = "pos") BlockPos pos) { + ((BlockParticleOptionExtension) original).fabric_setBlockPos(pos); return original; } } diff --git a/fabric-recipe-api-v1/src/main/java/net/fabricmc/fabric/mixin/recipe/sync/RecipeMapMixin.java b/fabric-recipe-api-v1/src/main/java/net/fabricmc/fabric/mixin/recipe/sync/RecipeMapMixin.java index 651d34af5c6..6dd8c88b847 100644 --- a/fabric-recipe-api-v1/src/main/java/net/fabricmc/fabric/mixin/recipe/sync/RecipeMapMixin.java +++ b/fabric-recipe-api-v1/src/main/java/net/fabricmc/fabric/mixin/recipe/sync/RecipeMapMixin.java @@ -57,12 +57,12 @@ private static void provideSerializerMap(Iterable> recipes, Call } @Inject(method = "create", at = @At(value = "INVOKE", target = "Lcom/google/common/collect/ImmutableMap$Builder;put(Ljava/lang/Object;Ljava/lang/Object;)Lcom/google/common/collect/ImmutableMap$Builder;")) - private static void fillSerializerMap(Iterable> recipes, CallbackInfoReturnable cir, @Local(name = "recipe") RecipeHolder entry, + private static void fillSerializerMap(Iterable> recipes, CallbackInfoReturnable cir, @Local(name = "recipe") RecipeHolder recipe, @Share("bySerializer") LocalRef, List>>> bySerializer) { - List> list = bySerializer.get().get(entry.value().getSerializer()); + List> list = bySerializer.get().get(recipe.value().getSerializer()); if (list != null) { - list.add(entry); + list.add(recipe); } } diff --git a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/particle/ScreenEffectRendererMixin.java b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/particle/ScreenEffectRendererMixin.java index e3a79878cd5..52e343b79c5 100644 --- a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/particle/ScreenEffectRendererMixin.java +++ b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/particle/ScreenEffectRendererMixin.java @@ -50,9 +50,9 @@ private static TextureAtlasSprite getParticleIconProxy(BlockModelShaper models, } @Inject(method = "getViewBlockingState", at = @At("RETURN")) - private static void onReturnGetInWallBlockState(CallbackInfoReturnable<@Nullable BlockState> cir, @Local(name = "testPos") BlockPos.MutableBlockPos mutable) { + private static void onReturnGetInWallBlockState(CallbackInfoReturnable<@Nullable BlockState> cir, @Local(name = "testPos") BlockPos.MutableBlockPos testPos) { if (cir.getReturnValue() != null) { - pos = mutable.immutable(); + pos = testPos.immutable(); } else { pos = null; } diff --git a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/ItemFrameRendererMixin.java b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/ItemFrameRendererMixin.java index a46311f1d23..501dc8c766f 100644 --- a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/ItemFrameRendererMixin.java +++ b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/ItemFrameRendererMixin.java @@ -34,9 +34,9 @@ abstract class ItemFrameRendererMixin { // Provide the BlockState as context. @Redirect(method = "submit(Lnet/minecraft/client/renderer/entity/state/ItemFrameRenderState;Lcom/mojang/blaze3d/vertex/PoseStack;Lnet/minecraft/client/renderer/SubmitNodeCollector;Lnet/minecraft/client/renderer/state/CameraRenderState;)V", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/renderer/SubmitNodeCollector;submitBlockModel(Lcom/mojang/blaze3d/vertex/PoseStack;Lnet/minecraft/client/renderer/rendertype/RenderType;Lnet/minecraft/client/renderer/block/model/BlockStateModel;FFFIII)V")) - private void renderProxy(SubmitNodeCollector submitNodeCollector, PoseStack poseStack, RenderType renderType, BlockStateModel model, float r, float g, float b, int light, int overlay, int outlineColor, @Local(name = "fakeBlockState") BlockState blockState) { + private void renderProxy(SubmitNodeCollector submitNodeCollector, PoseStack poseStack, RenderType renderType, BlockStateModel model, float r, float g, float b, int light, int overlay, int outlineColor, @Local(name = "fakeBlockState") BlockState fakeBlockState) { // The vertex consumer is for a special layer that renders solid, but vanilla has no equivalent // cutout/translucent layers that we can use here without risking compatibility. - submitNodeCollector.submitBlockStateModel(poseStack, blockLayer -> renderType, model, r, g, b, light, overlay, outlineColor, EmptyBlockAndTintGetter.INSTANCE, BlockPos.ZERO, blockState); + submitNodeCollector.submitBlockStateModel(poseStack, blockLayer -> renderType, model, r, g, b, light, overlay, outlineColor, EmptyBlockAndTintGetter.INSTANCE, BlockPos.ZERO, fakeBlockState); } } diff --git a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/SnowGolemHeadLayerMixin.java b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/SnowGolemHeadLayerMixin.java index 52ff3dbd4f9..789791f7f17 100644 --- a/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/SnowGolemHeadLayerMixin.java +++ b/fabric-renderer-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/renderer/block/render/SnowGolemHeadLayerMixin.java @@ -36,14 +36,14 @@ @Mixin(SnowGolemHeadLayer.class) abstract class SnowGolemHeadLayerMixin { @Redirect(method = "submit(Lcom/mojang/blaze3d/vertex/PoseStack;Lnet/minecraft/client/renderer/SubmitNodeCollector;ILnet/minecraft/client/renderer/entity/state/SnowGolemRenderState;FF)V", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/renderer/SubmitNodeCollector;submitBlockModel(Lcom/mojang/blaze3d/vertex/PoseStack;Lnet/minecraft/client/renderer/rendertype/RenderType;Lnet/minecraft/client/renderer/block/model/BlockStateModel;FFFIII)V")) - private void renderProxy(SubmitNodeCollector submitNodeCollector, PoseStack poseStack, RenderType renderType, BlockStateModel model, float r, float g, float b, int light, int overlay, int outlineColor, @Local(argsOnly = true) SnowGolemRenderState renderState, @Local(name = "pumpkinBlockState") BlockState blockState) { + private void renderProxy(SubmitNodeCollector submitNodeCollector, PoseStack poseStack, RenderType renderType, BlockStateModel model, float r, float g, float b, int light, int overlay, int outlineColor, @Local(argsOnly = true) SnowGolemRenderState renderState, @Local(name = "pumpkinBlockState") BlockState pumpkinBlockState) { // If true, the chunk layer is an outline chunk layer, and we want all geometry to use this chunk layer. if (renderState.appearsGlowing() && renderState.isInvisible) { // Fix tinted quads being rendered completely black and provide the BlockState as context. - submitNodeCollector.submitBlockStateModel(poseStack, _ -> renderType, model, 1, 1, 1, light, overlay, outlineColor, EmptyBlockAndTintGetter.INSTANCE, BlockPos.ZERO, blockState); + submitNodeCollector.submitBlockStateModel(poseStack, _ -> renderType, model, 1, 1, 1, light, overlay, outlineColor, EmptyBlockAndTintGetter.INSTANCE, BlockPos.ZERO, pumpkinBlockState); } else { // Support multi-chunk layer models, fix tinted quads being rendered completely black, and provide the BlockState as context. - submitNodeCollector.submitBlockStateModel(poseStack, ChunkSectionLayerHelper::getEntityBlockLayer, model, 1, 1, 1, light, overlay, outlineColor, EmptyBlockAndTintGetter.INSTANCE, BlockPos.ZERO, blockState); + submitNodeCollector.submitBlockStateModel(poseStack, ChunkSectionLayerHelper::getEntityBlockLayer, model, 1, 1, 1, light, overlay, outlineColor, EmptyBlockAndTintGetter.INSTANCE, BlockPos.ZERO, pumpkinBlockState); } } } diff --git a/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/ItemStackRenderStateMixin.java b/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/ItemStackRenderStateMixin.java index 4cea9d9f990..7518fbf6d2b 100644 --- a/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/ItemStackRenderStateMixin.java +++ b/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/ItemStackRenderStateMixin.java @@ -38,8 +38,8 @@ @Mixin(ItemStackRenderState.class) abstract class ItemStackRenderStateMixin { @Inject(method = "visitExtents(Ljava/util/function/Consumer;)V", at = @At(value = "NEW", target = "com/mojang/blaze3d/vertex/PoseStack$Pose")) - private void afterInitVecLoad(Consumer posConsumer, CallbackInfo ci, @Local(name = "scratch") Vector3f vec, @Share("pipe") LocalRef pipeRef) { - pipeRef.set(new QuadToPosPipe(posConsumer, vec)); + private void afterInitVecLoad(Consumer posConsumer, CallbackInfo ci, @Local(name = "scratch") Vector3f scratch, @Share("pipe") LocalRef pipeRef) { + pipeRef.set(new QuadToPosPipe(posConsumer, scratch)); } @Inject(method = "visitExtents(Ljava/util/function/Consumer;)V", at = @At(value = "INVOKE", target = "Lcom/mojang/blaze3d/vertex/PoseStack$Pose;setIdentity()V")) diff --git a/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/SectionCompilerMixin.java b/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/SectionCompilerMixin.java index 51c58b065aa..9705878dfe3 100644 --- a/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/SectionCompilerMixin.java +++ b/fabric-renderer-indigo/src/client/java/net/fabricmc/fabric/mixin/client/indigo/renderer/SectionCompilerMixin.java @@ -82,14 +82,14 @@ protected abstract BufferBuilder getOrBeginLayer( private void hookBuild(SectionPos sectionPos, RenderSectionRegion region, VertexSorting sorter, SectionBufferBuilderPack buffers, CallbackInfoReturnable cir, - @Local(name = "minPos") BlockPos sectionOrigin, + @Local(name = "minPos") BlockPos minPos, @Local(name = "poseStack") PoseStack poseStack, - @Local(name = "startedLayers") Map builderMap, + @Local(name = "startedLayers") Map startedLayers, @Local(name = "random") RandomSource random) { // hook just before iterating over the render chunk's blocks to capture the buffer builder map TerrainRenderContext renderer = TerrainRenderContext.POOL.get(); - renderer.prepare(region, sectionOrigin, - poseStack, random, layer -> getOrBeginLayer(builderMap, + renderer.prepare(region, minPos, + poseStack, random, layer -> getOrBeginLayer(startedLayers, buffers, layer)); ((AccessRenderSectionRegion) region).fabric_setRenderer(renderer); } @@ -108,12 +108,12 @@ private void hookBuild(SectionPos sectionPos, RenderSectionRegion region, Vertex * which was specifically created to provide for enhanced terrain rendering. */ @Redirect(method = "compile", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/state/BlockState;getRenderShape()Lnet/minecraft/world/level/block/RenderShape;")) - private RenderShape hookBuildRenderBlock(BlockState blockState, SectionPos sectionPos, RenderSectionRegion renderRegion, VertexSorting vertexSorter, SectionBufferBuilderPack buffers, @Local(name = "pos") BlockPos blockPos) { + private RenderShape hookBuildRenderBlock(BlockState blockState, SectionPos sectionPos, RenderSectionRegion renderRegion, VertexSorting vertexSorter, SectionBufferBuilderPack buffers, @Local(name = "pos") BlockPos pos) { RenderShape renderShape = blockState.getRenderShape(); if (renderShape == RenderShape.MODEL) { BlockStateModel model = blockRenderer.getBlockModel(blockState); - ((AccessRenderSectionRegion) renderRegion).fabric_getRenderer().bufferModel(model, blockState, blockPos); + ((AccessRenderSectionRegion) renderRegion).fabric_getRenderer().bufferModel(model, blockState, pos); return RenderShape.INVISIBLE; // Cancel the vanilla logic } diff --git a/fabric-rendering-fluids-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/fluid/LiquidBlockRendererMixin.java b/fabric-rendering-fluids-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/fluid/LiquidBlockRendererMixin.java index e43c6e06006..0e6f6977ddf 100644 --- a/fabric-rendering-fluids-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/fluid/LiquidBlockRendererMixin.java +++ b/fabric-rendering-fluids-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/fluid/LiquidBlockRendererMixin.java @@ -146,7 +146,7 @@ public int modTintColor(int original, BlockAndTintGetter level, BlockPos pos, Ve private TextureAtlasSprite modifyOverlaySprite( TextureAtlasSprite waterOverlay, BlockAndTintGetter level, - @Local(name = "tPos") BlockPos neighborPos, + @Local(name = "tPos") BlockPos tPos, @Local(name = "isLava") boolean isLava, @Local(name = "flowingSprite") TextureAtlasSprite flowingSprite, @Share("useOverlay") LocalBooleanRef useOverlay @@ -154,7 +154,7 @@ private TextureAtlasSprite modifyOverlaySprite( final FluidRenderHandlerInfo info = FluidRenderingImpl.getCurrentInfo(); boolean hasOverlay = info.handler != null ? info.hasOverlay : !isLava; - Block neighborBlock = level.getBlockState(neighborPos).getBlock(); + Block neighborBlock = level.getBlockState(tPos).getBlock(); useOverlay.set(hasOverlay && FluidRenderHandlerRegistry.INSTANCE.isBlockTransparent(neighborBlock)); if (useOverlay.get()) { diff --git a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LayerDefinitionsMixin.java b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LayerDefinitionsMixin.java index 73bc191fd28..54e9fc7b5cb 100644 --- a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LayerDefinitionsMixin.java +++ b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LayerDefinitionsMixin.java @@ -36,13 +36,13 @@ @Mixin(LayerDefinitions.class) abstract class LayerDefinitionsMixin { @Inject(method = "createRoots", at = @At(value = "INVOKE", target = "Lcom/google/common/collect/ImmutableMap$Builder;build()Lcom/google/common/collect/ImmutableMap;")) - private static void registerExtraModelData(CallbackInfoReturnable> info, @Local(name = "result") ImmutableMap.Builder builder) { + private static void registerExtraModelData(CallbackInfoReturnable> info, @Local(name = "result") ImmutableMap.Builder result) { for (Map.Entry entry : ModelLayerImpl.PROVIDERS.entrySet()) { - builder.put(entry.getKey(), entry.getValue().createLayerDefinition()); + result.put(entry.getKey(), entry.getValue().createLayerDefinition()); } for (Map.Entry, ModelLayerRegistry.TexturedArmorModelSetProvider> entry : ModelLayerImpl.ARMOR_PROVIDERS.entrySet()) { - entry.getKey().putFrom(entry.getValue().createArmorModelSet(), builder); + entry.getKey().putFrom(entry.getValue().createArmorModelSet(), result); } } } diff --git a/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/PackMixin.java b/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/PackMixin.java index 54109c569e3..37b76705ef0 100644 --- a/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/PackMixin.java +++ b/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/PackMixin.java @@ -33,9 +33,9 @@ @Mixin(Pack.class) public class PackMixin { @ModifyVariable(method = "readPackMetadata", at = @At("STORE"), name = "overlaySet") - private static List applyOverlayConditions(List overlays, @Local(name = "pack") PackResources resourcePack) throws IOException { + private static List applyOverlayConditions(List overlays, @Local(name = "pack") PackResources pack) throws IOException { List appliedOverlays = new ArrayList<>(overlays); - OverlayConditionsMetadata overlayMetadata = resourcePack.getMetadataSection(OverlayConditionsMetadata.SERIALIZER); + OverlayConditionsMetadata overlayMetadata = pack.getMetadataSection(OverlayConditionsMetadata.SERIALIZER); if (overlayMetadata != null) { appliedOverlays.addAll(overlayMetadata.appliedOverlays()); diff --git a/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/RegistryLoadTaskPendingRegistrationMixin.java b/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/RegistryLoadTaskPendingRegistrationMixin.java index bc7b80325cf..f50bfc4bcff 100644 --- a/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/RegistryLoadTaskPendingRegistrationMixin.java +++ b/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/RegistryLoadTaskPendingRegistrationMixin.java @@ -35,8 +35,8 @@ @Mixin(RegistryLoadTask.PendingRegistration.class) public abstract class RegistryLoadTaskPendingRegistrationMixin { @Inject(method = "loadFromResource", at = @At(value = "INVOKE", target = "Lcom/mojang/serialization/Decoder;parse(Lcom/mojang/serialization/DynamicOps;Ljava/lang/Object;)Lcom/mojang/serialization/DataResult;"), cancellable = true) - private static void loadFromResource(Decoder elementDecoder, RegistryOps ops, ResourceKey elementKey, Resource thunk, CallbackInfoReturnable> cir, @Local(name = "json") JsonElement jsonElement) { - if (jsonElement.isJsonObject() && !ResourceConditionsImpl.applyResourceConditions(jsonElement.getAsJsonObject(), elementKey.registry().toString(), elementKey.identifier(), ops.lookupProvider)) { + private static void loadFromResource(Decoder elementDecoder, RegistryOps ops, ResourceKey elementKey, Resource thunk, CallbackInfoReturnable> cir, @Local(name = "json") JsonElement json) { + if (json.isJsonObject() && !ResourceConditionsImpl.applyResourceConditions(json.getAsJsonObject(), elementKey.registry().toString(), elementKey.identifier(), ops.lookupProvider)) { cir.setReturnValue(Either.right(ResourceConditionsImpl.DISABLED_RESOURCE_EXCEPTION)); } } diff --git a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/DataPackCommandMixin.java b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/DataPackCommandMixin.java index e156e38f321..cf56dc94997 100644 --- a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/DataPackCommandMixin.java +++ b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/DataPackCommandMixin.java @@ -62,7 +62,7 @@ private static Stream filterDisabledPackSuggestions(Stream instance, } @Inject(method = "getPack", at = @At(value = "INVOKE", target = "Ljava/util/Collection;contains(Ljava/lang/Object;)Z")) - private static void errorOnInternalPack(CommandContext context, String name, boolean enable, CallbackInfoReturnable cir, @Local(name = "pack") Pack profile) throws CommandSyntaxException { - if (((FabricPack) profile).fabric$isHidden()) throw INTERNAL_PACK_EXCEPTION.create(profile.getId()); + private static void errorOnInternalPack(CommandContext context, String name, boolean enable, CallbackInfoReturnable cir, @Local(name = "pack") Pack pack) throws CommandSyntaxException { + if (((FabricPack) pack).fabric$isHidden()) throw INTERNAL_PACK_EXCEPTION.create(pack.getId()); } } diff --git a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/PackRepositoryMixin.java b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/PackRepositoryMixin.java index c4e9366ea11..ff8da807413 100644 --- a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/PackRepositoryMixin.java +++ b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/PackRepositoryMixin.java @@ -84,23 +84,23 @@ public void construct(RepositorySource[] resourcePackProviders, CallbackInfo inf } @Inject(method = "rebuildSelected", at = @At(value = "INVOKE", target = "Lcom/google/common/collect/ImmutableList;copyOf(Ljava/util/Collection;)Lcom/google/common/collect/ImmutableList;")) - private void handleAutoEnableDisable(Collection enabledNames, CallbackInfoReturnable> cir, @Local(name = "selectedAndPresent") List enabledAfterFirstRun) { - ModPackResourcesUtil.refreshAutoEnabledPacks(enabledAfterFirstRun, this.available); + private void handleAutoEnableDisable(Collection enabledNames, CallbackInfoReturnable> cir, @Local(name = "selectedAndPresent") List selectedAndPresent) { + ModPackResourcesUtil.refreshAutoEnabledPacks(selectedAndPresent, this.available); } @Inject(method = "addPack", at = @At(value = "INVOKE", target = "Ljava/util/List;add(Ljava/lang/Object;)Z", shift = At.Shift.AFTER)) - private void handleAutoEnable(String profile, CallbackInfoReturnable cir, @Local(name = "selectedCopy") List newlyEnabled) { + private void handleAutoEnable(String profile, CallbackInfoReturnable cir, @Local(name = "selectedCopy") List selectedCopy) { if (ModResourcePackCreator.POST_CHANGE_HANDLE_REQUIRED.contains(profile)) { - ModPackResourcesUtil.refreshAutoEnabledPacks(newlyEnabled, this.available); + ModPackResourcesUtil.refreshAutoEnabledPacks(selectedCopy, this.available); } } @Inject(method = "removePack", at = @At(value = "INVOKE", target = "Ljava/util/List;remove(Ljava/lang/Object;)Z")) - private void handleAutoDisable(String profile, CallbackInfoReturnable cir, @Local(name = "selectedCopy") List enabled) { + private void handleAutoDisable(String profile, CallbackInfoReturnable cir, @Local(name = "selectedCopy") List selectedCopy) { if (ModResourcePackCreator.POST_CHANGE_HANDLE_REQUIRED.contains(profile)) { - Set currentlyEnabled = enabled.stream().map(Pack::getId).collect(Collectors.toSet()); - enabled.removeIf(p -> !((FabricPack) p).fabric$parentsEnabled(currentlyEnabled)); - LOGGER.debug("[Fabric] Internal pack auto-removed upon disabling {}, result: {}", profile, enabled.stream().map(Pack::getId).toList()); + Set currentlyEnabled = selectedCopy.stream().map(Pack::getId).collect(Collectors.toSet()); + selectedCopy.removeIf(p -> !((FabricPack) p).fabric$parentsEnabled(currentlyEnabled)); + LOGGER.debug("[Fabric] Internal pack auto-removed upon disabling {}, result: {}", profile, selectedCopy.stream().map(Pack::getId).toList()); } } } diff --git a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/ReloadableServerResourcesMixin.java b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/ReloadableServerResourcesMixin.java index 286ddc9db44..a97b442c8b7 100644 --- a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/ReloadableServerResourcesMixin.java +++ b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/ReloadableServerResourcesMixin.java @@ -52,12 +52,12 @@ private static List onSetupDataReloaders( List reloaders, @Local(argsOnly = true, name = "fullRegistries") ReloadableServerRegistries.LoadResult loadResult, @Local(argsOnly = true, name = "enabledFeatures") FeatureFlagSet featureSet, - @Local(name = "result") ReloadableServerResources dataPackContents + @Local(name = "result") ReloadableServerResources result ) { var list = new ArrayList<>(reloaders); list.addFirst( new SetupMarkerResourceReloader( - dataPackContents, + result, loadResult.lookupWithUpdatedTags(), featureSet ) diff --git a/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/CrafterBlockMixin.java b/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/CrafterBlockMixin.java index d48db8489de..98ebfe1ba8b 100644 --- a/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/CrafterBlockMixin.java +++ b/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/CrafterBlockMixin.java @@ -42,13 +42,13 @@ public class CrafterBlockMixin { // Inject after vanilla's attempts to insert the stack into an inventory. @Inject(method = "dispenseItem", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/item/ItemStack;isEmpty()Z")) - private void transferOrSpawnStack(ServerLevel level, BlockPos pos, CrafterBlockEntity blockEntity, ItemStack inputStack, BlockState state, RecipeHolder recipe, CallbackInfo ci, @Local(name = "direction") Direction direction, @Local(name = "into") Container inventory, @Local(name = "remaining") ItemStack itemStack) { - if (inventory != null) { + private void transferOrSpawnStack(ServerLevel level, BlockPos pos, CrafterBlockEntity blockEntity, ItemStack inputStack, BlockState state, RecipeHolder recipe, CallbackInfo ci, @Local(name = "direction") Direction direction, @Local(name = "into") Container into, @Local(name = "remaining") ItemStack remaining) { + if (into != null) { // Vanilla already found and tested an inventory, nothing else to do even if it failed to insert. return; } - if (itemStack.isEmpty()) { + if (remaining.isEmpty()) { // Nothing left to do, in theory should never get here. return; } @@ -58,10 +58,10 @@ private void transferOrSpawnStack(ServerLevel level, BlockPos pos, CrafterBlockE if (target != null) { // Attempt to move the entire stack, and decrement the size of success moves. try (Transaction transaction = Transaction.openOuter()) { - long moved = target.insert(ItemVariant.of(itemStack), inputStack.getCount(), transaction); + long moved = target.insert(ItemVariant.of(remaining), inputStack.getCount(), transaction); if (moved > 0) { - itemStack.shrink((int) moved); + remaining.shrink((int) moved); transaction.commit(); } } diff --git a/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/HopperBlockEntityMixin.java b/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/HopperBlockEntityMixin.java index 5defeac4e24..899598585fc 100644 --- a/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/HopperBlockEntityMixin.java +++ b/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/mixin/transfer/HopperBlockEntityMixin.java @@ -52,9 +52,9 @@ public class HopperBlockEntityMixin { method = "ejectItems", cancellable = true ) - private static void hookInsert(Level level, BlockPos pos, HopperBlockEntity blockEntity, CallbackInfoReturnable cir, @Local(name = "container") Container targetInventory) { + private static void hookInsert(Level level, BlockPos pos, HopperBlockEntity blockEntity, CallbackInfoReturnable cir, @Local(name = "container") Container container) { // Let vanilla handle the transfer if it found an inventory. - if (targetInventory != null) return; + if (container != null) return; // Otherwise inject our transfer logic. Direction direction = ((HopperBlockEntityMixin) (Object) blockEntity).facing; @@ -81,9 +81,9 @@ private static void hookInsert(Level level, BlockPos pos, HopperBlockEntity bloc method = "suckInItems(Lnet/minecraft/world/level/Level;Lnet/minecraft/world/level/block/entity/Hopper;)Z", cancellable = true ) - private static void hookExtract(Level level, Hopper hopper, CallbackInfoReturnable cir, @Local(name = "container") Container inputInventory) { + private static void hookExtract(Level level, Hopper hopper, CallbackInfoReturnable cir, @Local(name = "container") Container container) { // Let vanilla handle the transfer if it found an inventory. - if (inputInventory != null) return; + if (container != null) return; // Otherwise inject our transfer logic. BlockPos sourcePos = BlockPos.containing(hopper.getLevelX(), hopper.getLevelY() + 1.0D, hopper.getLevelZ()); From 75b59cba0be3d5e97d8188eb6295a531721942ce Mon Sep 17 00:00:00 2001 From: modmuss50 Date: Sat, 24 Jan 2026 16:43:34 +0000 Subject: [PATCH 5/6] Cleanup --- checkstyle.xml | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/checkstyle.xml b/checkstyle.xml index acaa08df136..517d0cb2972 100644 --- a/checkstyle.xml +++ b/checkstyle.xml @@ -284,18 +284,16 @@ - + - + From 6ecff5a810da2cc5734ba8b46bbaa1b5e09da061 Mon Sep 17 00:00:00 2001 From: modmuss50 Date: Wed, 28 Jan 2026 22:19:29 +0000 Subject: [PATCH 6/6] Prevent using name for args only locals --- checkstyle.xml | 14 ++++++++++++++ .../gametest/input/GlCommandEncoderMixin.java | 2 +- .../mixin/content/registry/AxeItemMixin.java | 2 +- .../content/registry/FuelValuesMixin.java | 2 +- .../mixin/datagen/TagsProviderMixin.java | 2 +- .../fabric/mixin/entity/event/MobMixin.java | 2 +- .../ServerGamePacketListenerImplMixin.java | 4 ++-- .../fabric/mixin/item/ItemStackMixin.java | 18 +++++++++--------- .../client/message/ChatListenerMixin.java | 2 +- .../message/ClientPacketListenerMixin.java | 4 ++-- .../mixin/networking/ConnectionMixin.java | 4 ++-- .../object/builder/EntityTypeBuilderMixin.java | 2 +- .../client/particle/TerrainParticleMixin.java | 2 +- .../mixin/registry/sync/RegistriesMixin.java | 4 ++-- .../mixin/client/rendering/CapeLayerMixin.java | 2 +- .../rendering/LivingEntityRendererMixin.java | 2 +- .../SimpleJsonResourceReloadListenerMixin.java | 2 +- .../ReloadableServerResourcesMixin.java | 4 ++-- .../resource/SimpleReloadInstanceMixin.java | 4 ++-- .../fabric/mixin/tag/TagLoaderMixin.java | 2 +- 20 files changed, 47 insertions(+), 33 deletions(-) diff --git a/checkstyle.xml b/checkstyle.xml index 517d0cb2972..0fbe4531d22 100644 --- a/checkstyle.xml +++ b/checkstyle.xml @@ -296,6 +296,20 @@ + + + + + + diff --git a/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/input/GlCommandEncoderMixin.java b/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/input/GlCommandEncoderMixin.java index 7e04f52f28e..5ba1cb78874 100644 --- a/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/input/GlCommandEncoderMixin.java +++ b/fabric-client-gametest-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/gametest/input/GlCommandEncoderMixin.java @@ -32,7 +32,7 @@ @Mixin(GlCommandEncoder.class) public class GlCommandEncoderMixin { @WrapOperation(method = "presentTexture", at = @At(value = "INVOKE", target = "Lcom/mojang/blaze3d/opengl/DirectStateAccess;blitFrameBuffers(IIIIIIIIIIII)V")) - private void blitFrameBuffer(DirectStateAccess manager, int readFramebuffer, int drawFramebuffer, int srcX0, int srcY0, int srcX1, int srcY1, int dstX0, int dstY0, int dstX1, int dstY1, int mask, int filter, Operation original, @Local(argsOnly = true, name = "textureView") GpuTextureView gpuTextureView) { + private void blitFrameBuffer(DirectStateAccess manager, int readFramebuffer, int drawFramebuffer, int srcX0, int srcY0, int srcX1, int srcY1, int dstX0, int dstY0, int dstX1, int dstY1, int mask, int filter, Operation original, @Local(argsOnly = true) GpuTextureView gpuTextureView) { if (gpuTextureView.texture() == Minecraft.getInstance().getMainRenderTarget().getColorTexture()) { WindowHooks window = ((WindowHooks) (Object) Minecraft.getInstance().getWindow()); dstX1 = window.fabric_getRealFramebufferWidth(); diff --git a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/AxeItemMixin.java b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/AxeItemMixin.java index e785e87d105..9d609e637de 100644 --- a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/AxeItemMixin.java +++ b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/AxeItemMixin.java @@ -33,7 +33,7 @@ @Mixin(AxeItem.class) public class AxeItemMixin { @ModifyArg(method = "getStripped", at = @At(value = "INVOKE", target = "Ljava/util/Optional;map(Ljava/util/function/Function;)Ljava/util/Optional;")) - private Function handleCustomStrippingBehavior(Function mapper, @Local(argsOnly = true, name = "state") BlockState state) { + private Function handleCustomStrippingBehavior(Function mapper, @Local(argsOnly = true) BlockState state) { StrippableBlockRegistry.StrippingTransformer transformer = StrippableBlockRegistryImpl.getTransformer(state.getBlock()); if (transformer != null) { diff --git a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/FuelValuesMixin.java b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/FuelValuesMixin.java index 904cf93a831..57da5804d33 100644 --- a/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/FuelValuesMixin.java +++ b/fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/mixin/content/registry/FuelValuesMixin.java @@ -49,7 +49,7 @@ public abstract class FuelValuesMixin { ), allow = 1 ) - private static FuelValues.Builder build(FuelValues.Builder builder, TagKey tag, Operation operation, @Local(argsOnly = true, name = "registries") HolderLookup.Provider registries, @Local(argsOnly = true, name = "enabledFeatures") FeatureFlagSet features, @Local(argsOnly = true, name = "baseUnit") int baseSmeltTime) { + private static FuelValues.Builder build(FuelValues.Builder builder, TagKey tag, Operation operation, @Local(argsOnly = true) HolderLookup.Provider registries, @Local(argsOnly = true) FeatureFlagSet features, @Local(argsOnly = true) int baseSmeltTime) { final var context = new FuelRegistryEventsContextImpl(registries, features, baseSmeltTime); FuelValueEvents.BUILD.invoker().build(builder, context); diff --git a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/TagsProviderMixin.java b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/TagsProviderMixin.java index ae1488d0764..c2afdebd622 100644 --- a/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/TagsProviderMixin.java +++ b/fabric-data-generation-api-v1/src/main/java/net/fabricmc/fabric/mixin/datagen/TagsProviderMixin.java @@ -68,7 +68,7 @@ private boolean addReplaced(boolean replaced, @Local(name = "builder") TagBuilde @SuppressWarnings("unchecked") @WrapOperation(method = "lambda$run$2", at = @At(value = "INVOKE", target = "Ljava/util/concurrent/CompletableFuture;allOf([Ljava/util/concurrent/CompletableFuture;)Ljava/util/concurrent/CompletableFuture;")) - private CompletableFuture addTagAliasGroupBuilders(CompletableFuture[] futures, Operation> original, @Local(argsOnly = true, name = "cache") CachedOutput writer) { + private CompletableFuture addTagAliasGroupBuilders(CompletableFuture[] futures, Operation> original, @Local(argsOnly = true) CachedOutput writer) { if ((Object) this instanceof FabricTagsProvider) { // Note: no pattern matching instanceof so that we can cast directly to FabricTagsProvider instead of a wildcard Map.AliasGroupBuilder> builders = ((FabricTagsProvider) (Object) this).getAliasGroupBuilders(); diff --git a/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/MobMixin.java b/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/MobMixin.java index 976afe76ab2..3e40abd1c43 100644 --- a/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/MobMixin.java +++ b/fabric-entity-events-v1/src/main/java/net/fabricmc/fabric/mixin/entity/event/MobMixin.java @@ -30,7 +30,7 @@ @Mixin(Mob.class) public class MobMixin { @ModifyArg(method = "convertTo(Lnet/minecraft/world/entity/EntityType;Lnet/minecraft/world/entity/ConversionParams;Lnet/minecraft/world/entity/EntitySpawnReason;Lnet/minecraft/world/entity/ConversionParams$AfterConversion;)Lnet/minecraft/world/entity/Mob;", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/level/ServerLevel;addFreshEntity(Lnet/minecraft/world/entity/Entity;)Z")) - private Entity afterEntityConverted(Entity converted, @Local(argsOnly = true, name = "params") ConversionParams conversionContext) { + private Entity afterEntityConverted(Entity converted, @Local(argsOnly = true) ConversionParams conversionContext) { ServerLivingEntityEvents.MOB_CONVERSION.invoker().onConversion((Mob) (Object) this, (Mob) converted, conversionContext); return converted; } diff --git a/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerGamePacketListenerImplMixin.java b/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerGamePacketListenerImplMixin.java index 45a1d8c36df..72b5705a509 100644 --- a/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerGamePacketListenerImplMixin.java +++ b/fabric-events-interaction-v0/src/main/java/net/fabricmc/fabric/mixin/event/interaction/ServerGamePacketListenerImplMixin.java @@ -53,7 +53,7 @@ private void tryPickItem(ItemStack stack) { } @WrapOperation(method = "handlePickItemFromBlock", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/level/block/state/BlockState;getCloneItemStack(Lnet/minecraft/world/level/LevelReader;Lnet/minecraft/core/BlockPos;Z)Lnet/minecraft/world/item/ItemStack;")) - public ItemStack onPickItemFromBlock(BlockState state, LevelReader level, BlockPos pos, boolean includeData, Operation operation, @Local(argsOnly = true, name = "packet") ServerboundPickItemFromBlockPacket packet) { + public ItemStack onPickItemFromBlock(BlockState state, LevelReader level, BlockPos pos, boolean includeData, Operation operation, @Local(argsOnly = true) ServerboundPickItemFromBlockPacket packet) { ItemStack stack = PlayerPickItemEvents.BLOCK.invoker().onPickItemFromBlock(player, pos, state, packet.includeData()); if (stack == null) { @@ -67,7 +67,7 @@ public ItemStack onPickItemFromBlock(BlockState state, LevelReader level, BlockP } @WrapOperation(method = "handlePickItemFromEntity", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/entity/Entity;getPickResult()Lnet/minecraft/world/item/ItemStack;")) - public ItemStack onPickItemFromEntity(Entity entity, Operation operation, @Local(argsOnly = true, name = "packet") ServerboundPickItemFromEntityPacket packet) { + public ItemStack onPickItemFromEntity(Entity entity, Operation operation, @Local(argsOnly = true) ServerboundPickItemFromEntityPacket packet) { ItemStack stack = PlayerPickItemEvents.ENTITY.invoker().onPickItemFromEntity(player, entity, packet.includeData()); if (stack == null) { diff --git a/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/ItemStackMixin.java b/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/ItemStackMixin.java index d67b2de3be4..87173471625 100644 --- a/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/ItemStackMixin.java +++ b/fabric-item-api-v1/src/main/java/net/fabricmc/fabric/mixin/item/ItemStackMixin.java @@ -64,7 +64,7 @@ public abstract class ItemStackMixin implements FabricItemStack { public abstract void shrink(int i); @WrapOperation(method = "hurtAndBreak(ILnet/minecraft/world/entity/LivingEntity;Lnet/minecraft/world/entity/EquipmentSlot;)V", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/item/ItemStack;hurtAndBreak(ILnet/minecraft/server/level/ServerLevel;Lnet/minecraft/server/level/ServerPlayer;Ljava/util/function/Consumer;)V")) - private void hookDamage(ItemStack instance, int amount, ServerLevel serverLevel, ServerPlayer serverPlayer, Consumer consumer, Operation original, @Local(argsOnly = true, name = "owner") LivingEntity entity, @Local(argsOnly = true, name = "slot") EquipmentSlot slot) { + private void hookDamage(ItemStack instance, int amount, ServerLevel serverLevel, ServerPlayer serverPlayer, Consumer consumer, Operation original, @Local(argsOnly = true) LivingEntity entity, @Local(argsOnly = true) EquipmentSlot slot) { CustomDamageHandler handler = ((ItemExtensions) getItem()).fabric_getCustomDamageHandler(); /* @@ -94,10 +94,10 @@ The other damage method (which original.call discards) handles the creative mode @ModifyArg(method = "addDetailsToTooltip", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/item/ItemStack;addToTooltip(Lnet/minecraft/core/component/DataComponentType;Lnet/minecraft/world/item/Item$TooltipContext;Lnet/minecraft/world/item/component/TooltipDisplay;Ljava/util/function/Consumer;Lnet/minecraft/world/item/TooltipFlag;)V")) private DataComponentType preAppendComponentTooltip( DataComponentType componentType, - @Local(argsOnly = true, name = "context") Item.TooltipContext context, - @Local(argsOnly = true, name = "display") TooltipDisplay displayComponent, - @Local(argsOnly = true, name = "tooltipFlag") TooltipFlag type, - @Local(argsOnly = true, name = "builder") Consumer componentConsumer, + @Local(argsOnly = true) Item.TooltipContext context, + @Local(argsOnly = true) TooltipDisplay displayComponent, + @Local(argsOnly = true) TooltipFlag type, + @Local(argsOnly = true) Consumer componentConsumer, @Share("index") LocalIntRef index ) { preAppendTooltip(componentType, context, displayComponent, componentConsumer, type, index); @@ -107,10 +107,10 @@ private DataComponentType preAppendComponentTooltip( @ModifyArg(method = "addDetailsToTooltip", at = @At(value = "INVOKE", target = "Lnet/minecraft/world/item/component/TooltipDisplay;shows(Lnet/minecraft/core/component/DataComponentType;)Z")) private DataComponentType preShouldDisplay( DataComponentType componentType, - @Local(argsOnly = true, name = "context") Item.TooltipContext context, - @Local(argsOnly = true, name = "display") TooltipDisplay displayComponent, - @Local(argsOnly = true, name = "tooltipFlag") TooltipFlag type, - @Local(argsOnly = true, name = "builder") Consumer componentConsumer, + @Local(argsOnly = true) Item.TooltipContext context, + @Local(argsOnly = true) TooltipDisplay displayComponent, + @Local(argsOnly = true) TooltipFlag type, + @Local(argsOnly = true) Consumer componentConsumer, @Share("index") LocalIntRef index ) { preAppendTooltip(componentType, context, displayComponent, componentConsumer, type, index); diff --git a/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ChatListenerMixin.java b/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ChatListenerMixin.java index ab16607a9c0..458ea6c4932 100644 --- a/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ChatListenerMixin.java +++ b/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ChatListenerMixin.java @@ -68,7 +68,7 @@ private void fabric_onChatMessage(Component message, @Nullable PlayerChatMessage } @Inject(method = "handleSystemMessage", at = @At("HEAD"), cancellable = true) - private void fabric_allowGameMessage(Component _message, boolean overlay, CallbackInfo ci, @Local(argsOnly = true, name = "message") LocalRef message) { + private void fabric_allowGameMessage(Component _message, boolean overlay, CallbackInfo ci, @Local(argsOnly = true) LocalRef message) { if (ClientReceiveMessageEvents.ALLOW_GAME.invoker().allowReceiveGameMessage(message.get(), overlay)) { message.set(ClientReceiveMessageEvents.MODIFY_GAME.invoker().modifyReceivedGameMessage(message.get(), overlay)); ClientReceiveMessageEvents.GAME.invoker().onReceiveGameMessage(message.get(), overlay); diff --git a/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ClientPacketListenerMixin.java b/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ClientPacketListenerMixin.java index 69b56f0766b..4a24de184e5 100644 --- a/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ClientPacketListenerMixin.java +++ b/fabric-message-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/message/ClientPacketListenerMixin.java @@ -34,7 +34,7 @@ @Mixin(value = ClientPacketListener.class, priority = 800) public abstract class ClientPacketListenerMixin { @Inject(method = "sendChat", at = @At("HEAD"), cancellable = true) - private void fabric_allowSendChatMessage(String _content, CallbackInfo ci, @Local(argsOnly = true, name = "content") LocalRef content) { + private void fabric_allowSendChatMessage(String _content, CallbackInfo ci, @Local(argsOnly = true) LocalRef content) { if (ClientSendMessageEvents.ALLOW_CHAT.invoker().allowSendChatMessage(content.get())) { content.set(ClientSendMessageEvents.MODIFY_CHAT.invoker().modifySendChatMessage(content.get())); ClientSendMessageEvents.CHAT.invoker().onSendChatMessage(content.get()); @@ -45,7 +45,7 @@ private void fabric_allowSendChatMessage(String _content, CallbackInfo ci, @Loca } @Inject(method = "sendCommand", at = @At("HEAD"), cancellable = true) - private void fabric_allowSendCommandMessage(String _command, CallbackInfo ci, @Local(argsOnly = true, name = "command") LocalRef command) { + private void fabric_allowSendCommandMessage(String _command, CallbackInfo ci, @Local(argsOnly = true) LocalRef command) { if (ClientSendMessageEvents.ALLOW_COMMAND.invoker().allowSendCommandMessage(command.get())) { command.set(ClientSendMessageEvents.MODIFY_COMMAND.invoker().modifySendCommandMessage(command.get())); ClientSendMessageEvents.COMMAND.invoker().onSendCommandMessage(command.get()); diff --git a/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/ConnectionMixin.java b/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/ConnectionMixin.java index 4a6a09d2dfe..9549e05ebe7 100644 --- a/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/ConnectionMixin.java +++ b/fabric-networking-api-v1/src/main/java/net/fabricmc/fabric/mixin/networking/ConnectionMixin.java @@ -94,7 +94,7 @@ private void disconnectAddon(CallbackInfo ci) { } @ModifyArg(method = "setupInboundProtocol", at = @At(value = "INVOKE", target = "Lio/netty/channel/Channel;writeAndFlush(Ljava/lang/Object;)Lio/netty/channel/ChannelFuture;")) - private Object injectFabricPacketSlitterHandlerInbound(Object transitioner, @Local(argsOnly = true, name = "protocol") ProtocolInfo protocolInfo) { + private Object injectFabricPacketSlitterHandlerInbound(Object transitioner, @Local(argsOnly = true) ProtocolInfo protocolInfo) { PayloadTypeRegistryImpl payloadTypeRegistry = PayloadTypeRegistryImpl.get(protocolInfo); if (payloadTypeRegistry == null) { @@ -108,7 +108,7 @@ private Object injectFabricPacketSlitterHandlerInbound(Object transitioner, @Loc } @ModifyArg(method = "setupOutboundProtocol", at = @At(value = "INVOKE", target = "Lio/netty/channel/Channel;writeAndFlush(Ljava/lang/Object;)Lio/netty/channel/ChannelFuture;")) - private Object injectFabricPacketSlitterHandlerOutbound(Object transitioner, @Local(argsOnly = true, name = "protocol") ProtocolInfo protocolInfo) { + private Object injectFabricPacketSlitterHandlerOutbound(Object transitioner, @Local(argsOnly = true) ProtocolInfo protocolInfo) { PayloadTypeRegistryImpl payloadTypeRegistry = PayloadTypeRegistryImpl.get(protocolInfo); if (payloadTypeRegistry == null) { diff --git a/fabric-object-builder-api-v1/src/main/java/net/fabricmc/fabric/mixin/object/builder/EntityTypeBuilderMixin.java b/fabric-object-builder-api-v1/src/main/java/net/fabricmc/fabric/mixin/object/builder/EntityTypeBuilderMixin.java index 97fdf7e7799..ffd4da2c2fc 100644 --- a/fabric-object-builder-api-v1/src/main/java/net/fabricmc/fabric/mixin/object/builder/EntityTypeBuilderMixin.java +++ b/fabric-object-builder-api-v1/src/main/java/net/fabricmc/fabric/mixin/object/builder/EntityTypeBuilderMixin.java @@ -100,7 +100,7 @@ private static EntityType castMob( } @WrapOperation(method = "build", at = @At(value = "INVOKE", target = "Lnet/minecraft/util/Util;fetchChoiceType(Lcom/mojang/datafixers/DSL$TypeReference;Ljava/lang/String;)Lcom/mojang/datafixers/types/Type;")) - private @Nullable Type allowNoModdedDatafixers(DSL.TypeReference typeReference, String id, Operation> original, @Local(argsOnly = true, name = "name") ResourceKey> resourceKey) { + private @Nullable Type allowNoModdedDatafixers(DSL.TypeReference typeReference, String id, Operation> original, @Local(argsOnly = true) ResourceKey> resourceKey) { if (!resourceKey.identifier().getNamespace().equals(Identifier.DEFAULT_NAMESPACE)) { // Don't try to resolve the choice type for modded entities. return null; diff --git a/fabric-particles-v1/src/client/java/net/fabricmc/fabric/mixin/client/particle/TerrainParticleMixin.java b/fabric-particles-v1/src/client/java/net/fabricmc/fabric/mixin/client/particle/TerrainParticleMixin.java index b84780c700b..36d8892d92c 100644 --- a/fabric-particles-v1/src/client/java/net/fabricmc/fabric/mixin/client/particle/TerrainParticleMixin.java +++ b/fabric-particles-v1/src/client/java/net/fabricmc/fabric/mixin/client/particle/TerrainParticleMixin.java @@ -58,7 +58,7 @@ private TerrainParticleMixin() { allow = 1, name = "blockState" ) - private BlockState removeUntintableParticles(BlockState state, @Local(argsOnly = true, name = "level") ClientLevel level, @Local(argsOnly = true, name = "pos") BlockPos blockPos) { + private BlockState removeUntintableParticles(BlockState state, @Local(argsOnly = true) ClientLevel level, @Local(argsOnly = true) BlockPos blockPos) { if (!ParticleRenderEvents.ALLOW_TERRAIN_PARTICLE_TINT.invoker().allowTerrainParticleTint(state, level, blockPos)) { // As of 1.20.1, vanilla hardcodes grass block particles to not get tinted. return Blocks.GRASS_BLOCK.defaultBlockState(); diff --git a/fabric-registry-sync-v0/src/main/java/net/fabricmc/fabric/mixin/registry/sync/RegistriesMixin.java b/fabric-registry-sync-v0/src/main/java/net/fabricmc/fabric/mixin/registry/sync/RegistriesMixin.java index 65ecfac7df1..9bc5bf5c407 100644 --- a/fabric-registry-sync-v0/src/main/java/net/fabricmc/fabric/mixin/registry/sync/RegistriesMixin.java +++ b/fabric-registry-sync-v0/src/main/java/net/fabricmc/fabric/mixin/registry/sync/RegistriesMixin.java @@ -31,7 +31,7 @@ @Mixin(Registries.class) public class RegistriesMixin { @ModifyReturnValue(method = "elementsDirPath", at = @At("RETURN")) - private static String prependDirectoryWithNamespace(String original, @Local(argsOnly = true, name = "registryKey") ResourceKey> registryRef) { + private static String prependDirectoryWithNamespace(String original, @Local(argsOnly = true) ResourceKey> registryRef) { Identifier id = registryRef.identifier(); if (!id.getNamespace().equals(Identifier.DEFAULT_NAMESPACE)) { @@ -42,7 +42,7 @@ private static String prependDirectoryWithNamespace(String original, @Local(args } @ModifyReturnValue(method = "tagsDirPath", at = @At("RETURN")) - private static String prependTagDirectoryWithNamespace(String original, @Local(argsOnly = true, name = "registryKey") ResourceKey> registryRef) { + private static String prependTagDirectoryWithNamespace(String original, @Local(argsOnly = true) ResourceKey> registryRef) { Identifier id = registryRef.identifier(); if (!id.getNamespace().equals(Identifier.DEFAULT_NAMESPACE)) { diff --git a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/CapeLayerMixin.java b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/CapeLayerMixin.java index 9240ce728d2..5a0d4e747d5 100644 --- a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/CapeLayerMixin.java +++ b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/CapeLayerMixin.java @@ -32,7 +32,7 @@ @Mixin(CapeLayer.class) public class CapeLayerMixin { @Inject(at = @At(value = "HEAD"), method = "submit(Lcom/mojang/blaze3d/vertex/PoseStack;Lnet/minecraft/client/renderer/SubmitNodeCollector;ILnet/minecraft/client/renderer/entity/state/AvatarRenderState;FF)V", cancellable = true) - public void injectCapeRenderCheck(PoseStack poseStack, SubmitNodeCollector submitNodeCollector, int i, AvatarRenderState avatarRenderState, float f, float g, CallbackInfo ci, @Local(argsOnly = true, name = "state") AvatarRenderState state) { + public void injectCapeRenderCheck(PoseStack poseStack, SubmitNodeCollector submitNodeCollector, int i, AvatarRenderState avatarRenderState, float f, float g, CallbackInfo ci, @Local(argsOnly = true) AvatarRenderState state) { if (!LivingEntityFeatureRenderEvents.ALLOW_CAPE_RENDER.invoker().allowCapeRender(state)) { ci.cancel(); } diff --git a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LivingEntityRendererMixin.java b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LivingEntityRendererMixin.java index 5cffccc4b4e..1b86dee224c 100644 --- a/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LivingEntityRendererMixin.java +++ b/fabric-rendering-v1/src/client/java/net/fabricmc/fabric/mixin/client/rendering/LivingEntityRendererMixin.java @@ -36,7 +36,7 @@ abstract class LivingEntityRendererMixin { method = "extractRenderState(Lnet/minecraft/world/entity/LivingEntity;Lnet/minecraft/client/renderer/entity/state/LivingEntityRenderState;F)V", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/renderer/entity/layers/HumanoidArmorLayer;shouldRender(Lnet/minecraft/world/item/ItemStack;Lnet/minecraft/world/entity/EquipmentSlot;)Z") ) - private boolean toggleDefaultHeadItem(ItemStack headStack, EquipmentSlot slot, Operation original, @Local(argsOnly = true, name = "entity") LivingEntity entity) { + private boolean toggleDefaultHeadItem(ItemStack headStack, EquipmentSlot slot, Operation original, @Local(argsOnly = true) LivingEntity entity) { // Return value: true if the item isn't rendered if (original.call(headStack, slot)) return true; ArmorRenderer renderer = ArmorRendererRegistryImpl.get(headStack.getItem()); diff --git a/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/SimpleJsonResourceReloadListenerMixin.java b/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/SimpleJsonResourceReloadListenerMixin.java index 30a0aa500b4..e2a7a8c2014 100644 --- a/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/SimpleJsonResourceReloadListenerMixin.java +++ b/fabric-resource-conditions-api-v1/src/main/java/net/fabricmc/fabric/mixin/resource/conditions/SimpleJsonResourceReloadListenerMixin.java @@ -48,7 +48,7 @@ public class SimpleJsonResourceReloadListenerMixin { @WrapOperation(method = "scanDirectory(Lnet/minecraft/server/packs/resources/ResourceManager;Lnet/minecraft/resources/FileToIdConverter;Lcom/mojang/serialization/DynamicOps;Lcom/mojang/serialization/Codec;Ljava/util/Map;)V", at = @At(value = "INVOKE", target = "Lcom/mojang/serialization/Codec;parse(Lcom/mojang/serialization/DynamicOps;Ljava/lang/Object;)Lcom/mojang/serialization/DataResult;")) private static DataResult applyResourceConditions(Codec instance, DynamicOps dynamicOps, Object object, Operation> original, - @Local(argsOnly = true, name = "lister") FileToIdConverter resourceFinder, + @Local(argsOnly = true) FileToIdConverter resourceFinder, @Local(name = "entry") Map.Entry entry) { final JsonElement resourceData = (JsonElement) object; RegistryOps.@Nullable RegistryInfoLookup registryInfo = null; diff --git a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/ReloadableServerResourcesMixin.java b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/ReloadableServerResourcesMixin.java index a97b442c8b7..a020deeff07 100644 --- a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/ReloadableServerResourcesMixin.java +++ b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/ReloadableServerResourcesMixin.java @@ -50,8 +50,8 @@ public class ReloadableServerResourcesMixin implements FabricDataResourceStoreHo ) private static List onSetupDataReloaders( List reloaders, - @Local(argsOnly = true, name = "fullRegistries") ReloadableServerRegistries.LoadResult loadResult, - @Local(argsOnly = true, name = "enabledFeatures") FeatureFlagSet featureSet, + @Local(argsOnly = true) ReloadableServerRegistries.LoadResult loadResult, + @Local(argsOnly = true) FeatureFlagSet featureSet, @Local(name = "result") ReloadableServerResources result ) { var list = new ArrayList<>(reloaders); diff --git a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/SimpleReloadInstanceMixin.java b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/SimpleReloadInstanceMixin.java index 1bb1ea55355..cbd9bb5600b 100644 --- a/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/SimpleReloadInstanceMixin.java +++ b/fabric-resource-loader-v1/src/main/java/net/fabricmc/fabric/mixin/resource/SimpleReloadInstanceMixin.java @@ -40,7 +40,7 @@ public class SimpleReloadInstanceMixin { target = "Lnet/minecraft/server/packs/resources/SimpleReloadInstance;of(Lnet/minecraft/server/packs/resources/ResourceManager;Ljava/util/List;Ljava/util/concurrent/Executor;Ljava/util/concurrent/Executor;Ljava/util/concurrent/CompletableFuture;)Lnet/minecraft/server/packs/resources/ReloadInstance;" ) ) - private static List sortSimple(List reloaders, @Local(argsOnly = true, name = "resourceManager") ResourceManager resourceManager) { + private static List sortSimple(List reloaders, @Local(argsOnly = true) ResourceManager resourceManager) { if (resourceManager instanceof FabricMultiPackResourceManager flrm) { return ResourceLoaderImpl.sort(flrm.fabric$getPackType(), reloaders); } @@ -55,7 +55,7 @@ private static List sortSimple(List sortProfiled(List reloaders, @Local(argsOnly = true, name = "resourceManager") ResourceManager resourceManager) { + private static List sortProfiled(List reloaders, @Local(argsOnly = true) ResourceManager resourceManager) { if (resourceManager instanceof FabricMultiPackResourceManager flrm) { return ResourceLoaderImpl.sort(flrm.fabric$getPackType(), reloaders); } diff --git a/fabric-tag-api-v1/src/main/java/net/fabricmc/fabric/mixin/tag/TagLoaderMixin.java b/fabric-tag-api-v1/src/main/java/net/fabricmc/fabric/mixin/tag/TagLoaderMixin.java index 77a7e48566c..52f7fe5dd0c 100644 --- a/fabric-tag-api-v1/src/main/java/net/fabricmc/fabric/mixin/tag/TagLoaderMixin.java +++ b/fabric-tag-api-v1/src/main/java/net/fabricmc/fabric/mixin/tag/TagLoaderMixin.java @@ -37,7 +37,7 @@ public class TagLoaderMixin { // Fixes a likely vanilla bug causing loot table tags to not get loaded. @WrapOperation(method = "loadTagsForRegistry(Lnet/minecraft/server/packs/resources/ResourceManager;Lnet/minecraft/core/WritableRegistry;)V", at = @At(value = "INVOKE", target = "Lnet/minecraft/tags/TagLoader;loadTagsForRegistry(Lnet/minecraft/server/packs/resources/ResourceManager;Lnet/minecraft/resources/ResourceKey;Lnet/minecraft/tags/TagLoader$ElementLookup;)Ljava/util/Map;")) - private static Map, List>> loadTagsForRegistry(ResourceManager manager, ResourceKey> registryKey, TagLoader.ElementLookup> lookup, Operation, List>>> original, @Local(argsOnly = true, name = "registry") WritableRegistry registry) { + private static Map, List>> loadTagsForRegistry(ResourceManager manager, ResourceKey> registryKey, TagLoader.ElementLookup> lookup, Operation, List>>> original, @Local(argsOnly = true) WritableRegistry registry) { Map, List>> tags = original.call(manager, registryKey, lookup); registry.bindTags(tags); return tags;