Move InputEvent firing to a new class, InputEvents

Author: kitlith

patchwork-events-input/src/main/java/net/patchworkmc/api/input/ForgeMouse.java

@@ -0,0 +1,26 @@
+/*
+ * Minecraft Forge, Patchwork Project
+ * Copyright (c) 2016-2020, 2019-2020
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation version 2.1
+ * of the License.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
+ */
+
+package net.patchworkmc.api.input;
+
+public interface ForgeMouse {
+	boolean isMiddleDown();
+	double getXVelocity();
+	double getYVelocity();
+}

patchwork-events-input/src/main/java/net/patchworkmc/impl/event/input/InputEvents.java

@@ -0,0 +1,48 @@
+/*
+ * Minecraft Forge, Patchwork Project
+ * Copyright (c) 2016-2020, 2019-2020
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation version 2.1
+ * of the License.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
+ */
+
+package net.patchworkmc.impl.event.input;
+
+import net.minecraftforge.client.event.InputEvent;
+import net.minecraftforge.common.MinecraftForge;
+import net.minecraftforge.eventbus.api.Event;
+
+import net.minecraft.client.Mouse;
+
+import net.patchworkmc.api.input.ForgeMouse;
+
+public class InputEvents {
+	public static void fireMouseInput(int button, int action, int mods) {
+		MinecraftForge.EVENT_BUS.post(new InputEvent.MouseInputEvent(button, action, mods));
+	}
+
+	public static void fireKeyInput(int key, int scanCode, int action, int modifiers) {
+		MinecraftForge.EVENT_BUS.post(new InputEvent.KeyInputEvent(key, scanCode, action, modifiers));
+	}
+
+	public static boolean onMouseScroll(Mouse mouseHelper, double scrollDelta) {
+		final Event event = new InputEvent.MouseScrollEvent(scrollDelta, mouseHelper.wasLeftButtonClicked(), ((ForgeMouse) mouseHelper).isMiddleDown(), mouseHelper.wasRightButtonClicked(), mouseHelper.getX(), mouseHelper.getY());
+
+		return MinecraftForge.EVENT_BUS.post(event);
+	}
+
+	public static boolean onRawMouseClicked(int button, int action, int mods) {
+		return MinecraftForge.EVENT_BUS.post(new InputEvent.RawMouseEvent(button, action, mods));
+	}
+}

patchwork-events-input/src/main/java/net/patchworkmc/mixin/event/input/MixinKeyboard.java

@@ -19,8 +19,6 @@
 
 package net.patchworkmc.mixin.event.input;
 
-import net.minecraftforge.client.event.InputEvent;
-import net.minecraftforge.common.MinecraftForge;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.Shadow;
 import org.spongepowered.asm.mixin.injection.At;
@@ -30,15 +28,17 @@
 import net.minecraft.client.Keyboard;
 import net.minecraft.client.MinecraftClient;
 
+import net.patchworkmc.impl.event.input.InputEvents;
+
 @Mixin(Keyboard.class)
 public abstract class MixinKeyboard {
 	@Shadow
 	MinecraftClient client;
 
 	@Inject(method = "onKey", at = @At("RETURN"))
-	private void fireKeyInput(long window, int key, int scancode, int i, int j, CallbackInfo info) {
+	private void fireKeyInput(long window, int key, int scancode, int action, int modifiers, CallbackInfo info) {
 		if (window == this.client.window.getHandle()) {
-			MinecraftForge.EVENT_BUS.post(new InputEvent.KeyInputEvent(key, scancode, i, j));
+			InputEvents.fireKeyInput(key, scancode, action, modifiers);
 		}
 	}
 }

patchwork-events-input/src/main/java/net/patchworkmc/mixin/event/input/MixinMouse.java

@@ -19,11 +19,7 @@
 
 package net.patchworkmc.mixin.event.input;
 
-import net.minecraftforge.client.event.InputEvent;
-import net.minecraftforge.common.MinecraftForge;
-import net.minecraftforge.eventbus.api.Event;
 import org.spongepowered.asm.mixin.Mixin;
-import org.spongepowered.asm.mixin.Shadow;
 import org.spongepowered.asm.mixin.injection.At;
 import org.spongepowered.asm.mixin.injection.At.Shift;
 import org.spongepowered.asm.mixin.injection.Inject;
@@ -32,36 +28,26 @@
 
 import net.minecraft.client.Mouse;
 
-@Mixin(Mouse.class)
-public abstract class MixinMouse {
-	@Shadow
-	boolean middleButtonClicked;
-	@Shadow
-	abstract boolean wasLeftButtonClicked();
-	@Shadow
-	abstract boolean wasRightButtonClicked();
-	@Shadow
-	abstract double getX();
-	@Shadow
-	abstract double getY();
+import net.patchworkmc.api.input.ForgeMouse;
+import net.patchworkmc.impl.event.input.InputEvents;
 
+@Mixin(Mouse.class)
+public abstract class MixinMouse implements ForgeMouse {
 	@Inject(method = "onMouseButton", at = @At("RETURN"), cancellable = true)
 	private void fireMouseInput(long window, int button, int action, int mods, CallbackInfo info) {
-		MinecraftForge.EVENT_BUS.post(new InputEvent.MouseInputEvent(button, action, mods));
+		InputEvents.fireMouseInput(button, action, mods);
 	}
 
 	@Inject(method = "onMouseButton", at = @At(value = "FIELD", ordinal = 3, target = "Lnet/minecraft/client/Mouse;client:Lnet/minecraft/client/MinecraftClient;", shift = Shift.BEFORE), cancellable = true)
 	private void onRawMouseClicked(long window, int button, int action, int mods, CallbackInfo info) {
-		if (MinecraftForge.EVENT_BUS.post(new InputEvent.RawMouseEvent(button, action, mods))) {
+		if (InputEvents.onRawMouseClicked(button, action, mods)) {
 			info.cancel();
 		}
 	}
 
 	@Inject(method = "onMouseScroll", locals = LocalCapture.CAPTURE_FAILHARD, at = @At(value = "INVOKE", target = "Lnet/minecraft/client/network/ClientPlayerEntity;isSpectator()Z", shift = Shift.BEFORE), cancellable = true)
-	private void onMouseScroll(long window, double d, double e, CallbackInfo info, double f, float i) {
-		final Event event = new InputEvent.MouseScrollEvent(f, wasLeftButtonClicked(), middleButtonClicked, wasRightButtonClicked(), getX(), getY());
-
-		if (MinecraftForge.EVENT_BUS.post(event)) {
+	private void onMouseScroll(long window, double d, double e, CallbackInfo info, double scrollDelta, float i) {
+		if (InputEvents.onMouseScroll((Mouse) (Object) this, scrollDelta)) {
 			info.cancel();
 		}
 	}