Feat[input]: GLFW direct controller input (#6604)

Author: artdeell

app_pojavlauncher/src/main/assets/components/arc_dns_injector/arc_dns_injector.jar

@@ -1 +1 @@
-1687691695196
+1738839797928

app_pojavlauncher/src/main/assets/components/arc_dns_injector/version

@@ -1 +1 @@
-1692525087345
+1738839797913

app_pojavlauncher/src/main/assets/components/forge_installer/forge_installer.jar

@@ -1 +1 @@
-1732218529630
+1738839797921

app_pojavlauncher/src/main/assets/components/forge_installer/version

@@ -29,6 +29,8 @@
 import net.kdt.pojavlaunch.customcontrols.ControlLayout;
 import net.kdt.pojavlaunch.customcontrols.gamepad.DefaultDataProvider;
 import net.kdt.pojavlaunch.customcontrols.gamepad.Gamepad;
+import net.kdt.pojavlaunch.customcontrols.gamepad.direct.DirectGamepad;
+import net.kdt.pojavlaunch.customcontrols.gamepad.direct.DirectGamepadEnableHandler;
 import net.kdt.pojavlaunch.customcontrols.mouse.AbstractTouchpad;
 import net.kdt.pojavlaunch.customcontrols.mouse.AndroidPointerCapture;
 import net.kdt.pojavlaunch.customcontrols.mouse.InGUIEventProcessor;
@@ -40,15 +42,16 @@
 
 import org.lwjgl.glfw.CallbackBridge;
 
+import fr.spse.gamepad_remapper.GamepadHandler;
 import fr.spse.gamepad_remapper.RemapperManager;
 import fr.spse.gamepad_remapper.RemapperView;
 
 /**
  * Class dealing with showing minecraft surface and taking inputs to dispatch them to minecraft
  */
-public class MinecraftGLSurface extends View implements GrabListener {
+public class MinecraftGLSurface extends View implements GrabListener, DirectGamepadEnableHandler {
     /* Gamepad object for gamepad inputs, instantiated on need */
-    private Gamepad mGamepad = null;
+    private GamepadHandler mGamepadHandler;
     /* The RemapperView.Builder object allows you to set which buttons to remap */
     private final RemapperManager mInputManager = new RemapperManager(getContext(), new RemapperView.Builder(null)
             .remapA(true)
@@ -88,6 +91,7 @@ public MinecraftGLSurface(Context context) {
     public MinecraftGLSurface(Context context, AttributeSet attributeSet) {
         super(context, attributeSet);
         setFocusable(true);
+        CallbackBridge.setDirectGamepadEnableHandler(this);
     }
 
     @RequiresApi(api = Build.VERSION_CODES.O)
@@ -203,7 +207,11 @@ public boolean onTouchEvent(MotionEvent e) {
     }
 
     private void createGamepad(View contextView, InputDevice inputDevice) {
-        mGamepad = new Gamepad(contextView, inputDevice, DefaultDataProvider.INSTANCE, true);
+        if(CallbackBridge.sGamepadDirectInput) {
+            mGamepadHandler = new DirectGamepad();
+        }else {
+            mGamepadHandler = new Gamepad(contextView, inputDevice, DefaultDataProvider.INSTANCE, true);
+        }
     }
 
     /**
@@ -215,9 +223,9 @@ public boolean dispatchGenericMotionEvent(MotionEvent event) {
         int mouseCursorIndex = -1;
 
         if(Gamepad.isGamepadEvent(event)){
-            if(mGamepad == null) createGamepad(this, event.getDevice());
+            if(mGamepadHandler == null) createGamepad(this, event.getDevice());
 
-            mInputManager.handleMotionEventInput(getContext(), event, mGamepad);
+            mInputManager.handleMotionEventInput(getContext(), event, mGamepadHandler);
             return true;
         }
 
@@ -287,9 +295,9 @@ public boolean processKeyEvent(KeyEvent event) {
         }
 
         if(Gamepad.isGamepadEvent(event)){
-            if(mGamepad == null) createGamepad(this, event.getDevice());
+            if(mGamepadHandler == null) createGamepad(this, event.getDevice());
 
-            mInputManager.handleKeyEventInput(getContext(), event, mGamepad);
+            mInputManager.handleKeyEventInput(getContext(), event, mGamepadHandler);
             return true;
         }
 
@@ -411,6 +419,17 @@ private void updateGrabState(boolean isGrabbing) {
         }
     }
 
+    @Override
+    public void onDirectGamepadEnabled() {
+        post(()->{
+            if(mGamepadHandler != null && mGamepadHandler instanceof Gamepad) {
+                ((Gamepad)mGamepadHandler).removeSelf();
+            }
+            // Force gamepad recreation on next event
+            mGamepadHandler = null;
+        });
+    }
+
     /** A small interface called when the listener is ready for the first time */
     public interface SurfaceReadyListener {
         void isReady();

app_pojavlauncher/src/main/assets/components/lwjgl3/lwjgl-glfw-classes.jar

@@ -30,4 +30,9 @@ public boolean isGrabbing() {
     public void attachGrabListener(GrabListener grabListener) {
         CallbackBridge.addGrabListener(grabListener);
     }
+
+    @Override
+    public void detachGrabListener(GrabListener grabListener) {
+        CallbackBridge.removeGrabListener(grabListener);
+    }
 }

app_pojavlauncher/src/main/assets/components/lwjgl3/version

@@ -26,7 +26,6 @@
 import net.kdt.pojavlaunch.GrabListener;
 import net.kdt.pojavlaunch.LwjglGlfwKeycode;
 import net.kdt.pojavlaunch.R;
-import net.kdt.pojavlaunch.prefs.LauncherPreferences;
 import net.kdt.pojavlaunch.utils.MCOptionUtils;
 
 import org.lwjgl.glfw.CallbackBridge;
@@ -88,6 +87,8 @@ public class Gamepad implements GrabListener, GamepadHandler {
 
     private final GamepadDataProvider mMapProvider;
 
+    private boolean mRemoved = false;
+
     public Gamepad(View contextView, InputDevice inputDevice, GamepadDataProvider mapProvider, boolean showCursor){
         Settings.setDeadzoneScale(PREF_DEADZONE_SCALE);
 
@@ -96,7 +97,7 @@ public Gamepad(View contextView, InputDevice inputDevice, GamepadDataProvider ma
             @Override
             public void doFrame(long frameTimeNanos) {
                 tick(frameTimeNanos);
-                mScreenChoreographer.postFrameCallback(this);
+                if(!mRemoved) mScreenChoreographer.postFrameCallback(this);
             }
         };
         mScreenChoreographer.postFrameCallback(frameCallback);
@@ -453,4 +454,15 @@ public void handleGamepadInput(int keycode, float value) {
                 break;
         }
     }
+
+    /**
+     * Stops the Gamepad and removes all traces of the Gamepad from the view hierarchy.
+     * After this call, the Gamepad is not recoverable and a new one must be made.
+     */
+    public void removeSelf() {
+        mRemoved = true;
+        mMapProvider.detachGrabListener(this);
+        ViewGroup viewGroup = (ViewGroup) mPointerImageView.getParent();
+        if(viewGroup != null) viewGroup.removeView(mPointerImageView);
+    }
 }

app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/MinecraftGLSurface.java

@@ -7,4 +7,5 @@ public interface GamepadDataProvider {
     GamepadMap getGameMap();
     boolean isGrabbing();
     void attachGrabListener(GrabListener grabListener);
+    void detachGrabListener(GrabListener grabListener);
 }