Skip to content
This repository was archived by the owner on Feb 19, 2019. It is now read-only.

Commit efa2913

Browse files
author
LeafHacker
committed
Attempt to fix weird error with @invoker
For some reason the following error occured when we used public @Invokers in MixinMinecraft MixinMinecraft.java:59: error: Mapping conflict for @invoker target clickMouse: func_147116_af()V for target net.minecraft.client.Minecraft conflicts with existing mapping Lnet/minecraft/client/Minecraft;func_147116_af()V @invoker("clickMouse") @OverRide public abstract void leftClickMouse(); ^
1 parent 2924358 commit efa2913

File tree

2 files changed

+5
-20
lines changed

2 files changed

+5
-20
lines changed

src/main/java/me/zero/client/load/mixin/MixinMinecraft.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,8 @@
3131
import net.minecraft.util.Session;
3232
import net.minecraft.util.Timer;
3333
import org.spongepowered.asm.mixin.Mixin;
34+
import org.spongepowered.asm.mixin.Shadow;
3435
import org.spongepowered.asm.mixin.gen.Accessor;
35-
import org.spongepowered.asm.mixin.gen.Invoker;
3636
import org.spongepowered.asm.mixin.injection.At;
3737
import org.spongepowered.asm.mixin.injection.Inject;
3838
import org.spongepowered.asm.mixin.injection.ModifyVariable;
@@ -56,15 +56,15 @@ public abstract class MixinMinecraft implements IMinecraft {
5656
@Accessor @Override public abstract void setSession(Session session);
5757
@Accessor @Override public abstract void setRightClickDelayTimer(int delay);
5858

59-
@Invoker("clickMouse") @Override public abstract void leftClickMouse();
60-
@Invoker("rightClickMouse") @Override public abstract void rightClickMouse();
61-
@Invoker("middleClickMouse") @Override public abstract void middleClickMouse();
59+
@Shadow private void clickMouse() {}
60+
@Shadow private void rightClickMouse() {}
61+
@Shadow private void middleClickMouse() {}
6262

6363
@Override
6464
public void clickMouse(ClickEvent.MouseButton button) {
6565
// IF statements are required because Mixin doesn't support SWITCH
6666
if (button == LEFT)
67-
leftClickMouse();
67+
clickMouse();
6868
if (button == RIGHT)
6969
rightClickMouse();
7070
if (button == MIDDLE)

src/main/java/me/zero/client/load/mixin/wrapper/IMinecraft.java

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -52,19 +52,4 @@ public interface IMinecraft {
5252
*/
5353
void clickMouse(ClickEvent.MouseButton button);
5454

55-
/**
56-
* Click the left mouse button
57-
*/
58-
void leftClickMouse();
59-
60-
/**
61-
* Click the right mouse button
62-
*/
63-
void rightClickMouse();
64-
65-
/**
66-
* Click the middle mouse button
67-
*/
68-
void middleClickMouse();
69-
7055
}

0 commit comments

Comments
 (0)