[WIP DRAFT] Add game specific mappings to suit gamepads, Add XBox style controll layout, Upped version

[BlazeEntertainment]

• Added Game specific mappings. In games like Night Stalker, you need to use the keypad to fire the weapon which is near impossible when using a keypad or keyboard. So wrote up specific key mappings to face buttons that make sense.
• Added XBOX style controller prompts to in game menu. A simple ifdef that can be enabled to flip the face buttons if the device uses XBOX style face button layouts instead of RetroPad
• Upped version to 1.2, because we added savestates and overhauled a number of elements and fixed a number of graphical, sound and accuracy issues ontop of the WIP mappings.

Todo

• Some games won't work well with modern game controllers without a keypad. These games have been remapped to facebuttons to be functional like Night Stalker, however these fixes are currently achieved with #IFDEF's. Need to replace these IFDEFS with some form of decent game detection so it automatically sets these new mappings.

[markwkidd]

The statement about controllers not using the retropad doesn't make sense. All libretro controller input is via the retropad, and Xbox controllers can be mapped to the retropad as well as any other.

…

On Thu, Jul 1, 2021, 1:15 PM BlazeEntertainment ***@***.***> wrote: - Added Game specific mappings. In games like Night Stalker, you need to use the keypad to fire the weapon which is near impossible when using a keypad or keyboard. So wrote up specific key mappings to face buttons that make sense. - Added XBOX style controller prompts to in game menu. A simple ifdef that can be enabled to flip the face buttons if the device uses XBOX style face button layouts instead of RetroPad - Upped version to 1.2, because we added savestates and overhauled a number of elements and fixed a number of graphical, sound and accuracy issues ontop of the WIP mappings. Todo - Some games won't work well with modern game controllers without a keypad. These games have been remapped to facebuttons to be functional like Night Stalker, however these fixes are currently achieved with #IFDEF's. Need to replace these IFDEFS with some form of decent game detection so it automatically sets these new mappings. ------------------------------ You can view, comment on, or merge this pull request online at: #60 Commit Summary - Fix controls for some games, bump up version, add to-do - Fix slapshot instructions - Update and fix slapshot mapping - Remove leftover Makefile modification File Changes - *M* README.md <https://github.com/libretro/FreeIntv/pull/60/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5> (3) - *M* src/controller.c <https://github.com/libretro/FreeIntv/pull/60/files#diff-fe98f6a7ce840f4d3d44cd630b5cf26625d82041ff84d0f6da3afa76fac5bf75> (40) - *M* src/libretro.c <https://github.com/libretro/FreeIntv/pull/60/files#diff-72aff6a6a6f37ebdd966bafa1ed685c9154866bb96b84b37e8aa25ccef7074a7> (56) Patch Links: - https://github.com/libretro/FreeIntv/pull/60.patch - https://github.com/libretro/FreeIntv/pull/60.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#60>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEVGC5V4FWZ4FBHKB3MLZELTVSPCNANCNFSM47VEIKUA> .

[BlazeEntertainment]

Appologies @markwkidd, I worded the MR extremely badly. I messaged you in person however please ignore this commit for now as upon reflection I want to reword some of these changes to make more sense and add some of the menu layout changes as a Core Option instead of an ifdef.

[inactive123]

I will have to think some more about this. I think it'd be a fundamental mistake to start dirtying up the codebase with all sorts of ifdefs for certain controllers. Best there is some libretro API extension instead to accomodate something like that. Anyway, we have to put this on hold for now until we can internally think of some solution.

[lgtm-com]

This pull request fixes 3 alerts when merging a18e0b1 into 5fc8d85 - view on LGTM.com

fixed alerts:

• 3 for Short global name

[lgtm-com]

This pull request fixes 4 alerts when merging 87fe95e into 5fc8d85 - view on LGTM.com

fixed alerts:

• 4 for Short global name