Input: Add SDL_GameController Backend

[Ryzee119]

I wrote a SDL_GameController backend for a port I am working on.

I appreciate there is probably many ways of doing this so will leave up to the author of this repo if they would like to merge or they had something else in mind. Happy to make changes if requested.

Benefits of this way:

• SDL_Gamecontroller provides a consistent button mapping across a range of controllers
• gamecontroller_to_keyboard() in input.c has been added to convert button events to SDL_Keycodes to integrate with the existing code base as much as possible.
• Keyboard input game controller input can both work at the same time

Current the mapping is as follows but happy to change based on feedback

• SDL_CONTROLLER_BUTTON_A = cfg_jump_key
• SDL_CONTROLLER_BUTTON_X/B = cfg_bomb_key
• SDL_CONTROLLER_BUTTON_DPAD = Arrow keys
• SDL_CONTROLLER_BUTTON_START = ENTER
• SDL_CONTROLLER_BUTTON_BACK= q (quit_game_dialog)
• SDL_CONTROLLER_BUTTON_Y = ESCAPE (help_menu_dialog)

What it doesn't do:

• Button mapping is hardcoded. The menu option to redefine joystick mapping still isnt enabled (SDL_Gamecontroller should provide a fairly consistent experience)
• No way of entering text for high score screen etc. This is how the original game would have behaved.
• In game dialog text still shows keyboard mapping. This is how the original game would have behaved I think?

I have tested with my wired Xbox360 controller.

This will potentially close issue #12

[BeWorld2018]

hi

Work fine, thanks. Maybe you can add Axis, i create it working in game only (not use your function) if (event.type == SDL_CONTROLLERAXISMOTION) { switch (event.caxis.axis) { case SDL_CONTROLLER_AXIS_LEFTX: case SDL_CONTROLLER_AXIS_RIGHTX: if (event.caxis.value < -16384) { left_key_pressed = 1; } else { left_key_pressed = 0; } if (event.caxis.value > 16384) { right_key_pressed = 1; } else { right_key_pressed = 0; } break; case SDL_CONTROLLER_AXIS_LEFTY: case SDL_CONTROLLER_AXIS_RIGHTY: if (event.caxis.value < -16384) { input_up_key_pressed = 1; } else { input_up_key_pressed = 0; } if (event.caxis.value > 16384) { down_key_pressed = 1; } else { down_key_pressed = 0; } break; } } You can close gamepad and maybe SubSystem for better integration ?

void gamepad_shutdown() {
    if (gamepad != NULL) {
        SDL_GameControllerClose(gamepad );
    }
        SDL_QuitSubSystem(SDL_INIT_GAMECONTROLLER);
}

thanks

[Ryzee119]

Thanks :smile: I'll add the deinit for sure and see what the best way of integrating the axis code is. I should be able to use your code snippet or close to it

[yuv422]

Hi Ryzee119, I'm currently working on controller support too. So I probably won't merge this PR. I'm refactoring the input handling a bit to get rid of key repeat and to implement controller mapping via the redefine menu.

[Ryzee119]

No problem. Thanks for your work on this awesome game :)

I'll keep an eye out for future updates