Remove duplicate MovementInput implementation, Fixes #312

src/main/java/baritone/bot/spec/BotMovementInput.java

@@ -1,71 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone 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, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone 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 Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.bot.spec;
-
-import baritone.api.utils.IInputOverrideHandler;
-import baritone.api.utils.input.Input;
-import baritone.bot.IBaritoneUser;
-import net.minecraft.util.MovementInput;
-
-/**
- * @author Brady
- * @since 10/29/2018
- */
-public class BotMovementInput extends MovementInput {
-
-    private final IBaritoneUser user;
-
-    public BotMovementInput(IBaritoneUser user) {
-        this.user = user;
-    }
-
-    public void updatePlayerMoveState() {
-        this.moveStrafe = 0.0F;
-        this.moveForward = 0.0F;
-
-        // These are placeholders until an input overrider is implemented for bots
-        IInputOverrideHandler i = user.getBaritone().getInputOverrideHandler();
-        boolean forward = i.isInputForcedDown(Input.MOVE_FORWARD);
-        boolean back = i.isInputForcedDown(Input.MOVE_BACK);
-        boolean left = i.isInputForcedDown(Input.MOVE_LEFT);
-        boolean right = i.isInputForcedDown(Input.MOVE_RIGHT);
-        jump = i.isInputForcedDown(Input.JUMP); // oppa
-        boolean sneak = i.isInputForcedDown(Input.SNEAK);
-
-        if (this.forwardKeyDown = forward) {
-            this.moveForward++;
-        }
-
-        if (this.backKeyDown = back) {
-            this.moveForward--;
-        }
-
-        if (this.leftKeyDown = left) {
-            this.moveStrafe++;
-        }
-
-        if (this.rightKeyDown = right) {
-            this.moveStrafe--;
-        }
-
-        if (this.sneak = sneak) {
-            this.moveStrafe *= 0.3D;
-            this.moveForward *= 0.3D;
-        }
-    }
-}

src/main/java/baritone/bot/spec/EntityBot.java

@@ -18,6 +18,7 @@
 package baritone.bot.spec;
 
 import baritone.bot.IBaritoneUser;
+import baritone.utils.PlayerMovementInput;
 import net.minecraft.client.Minecraft;
 import net.minecraft.client.entity.EntityPlayerSP;
 import net.minecraft.client.network.NetHandlerPlayClient;
@@ -66,7 +67,7 @@ public class EntityBot extends EntityPlayerSP {
     public EntityBot(IBaritoneUser user, Minecraft mc, World world, NetHandlerPlayClient netHandlerPlayClient, StatisticsManager statisticsManager, RecipeBook recipeBook) {
         super(mc, world, netHandlerPlayClient, statisticsManager, recipeBook);
         this.user = user;
-        this.movementInput = new BotMovementInput(this.user);
+        this.movementInput = new PlayerMovementInput(this.user.getBaritone().getInputOverrideHandler());
     }
 
     @Override

src/main/java/baritone/utils/PlayerMovementInput.java

@@ -17,13 +17,15 @@
 
 package baritone.utils;
 
+import baritone.api.utils.IInputOverrideHandler;
 import baritone.api.utils.input.Input;
 import net.minecraft.util.MovementInput;
 
 public class PlayerMovementInput extends MovementInput {
-    private final InputOverrideHandler handler;
 
-    public PlayerMovementInput(InputOverrideHandler handler) {
+    private final IInputOverrideHandler handler;
+
+    public PlayerMovementInput(IInputOverrideHandler handler) {
         this.handler = handler;
     }