RSDKModding / RSDKv4-Decompilation

A complete decompilation of Sonic 1 & Sonic 2 (2013) & Retro Engine (v4)
Other
937 stars 175 forks source link

[Android/Linux] Input broken as of fd5f527 #357

Closed SeongGino closed 1 year ago

SeongGino commented 2 years ago

Technically a more detailed synopsis/duplicate of #354.

After recompiling my Android builds in anticipation of Sonic 1 Forever 1.4.2, I've noticed that the current master (9fffd6b) does not seem to recognize the physical gamepad input on the game's (S1 and S2 vanilla) title screens, but works in the start menu frontend.

After bisecting, 559adb1 is the last version to accept input properly, with fd5f527 being the commit that apparently breaks input on vanilla game title screens, and all the menus of S1F and Sonic 2 Absolute are inaccessible.

Using the dev menu to spawn into a stage proper, the controls do work on affected commits*; implying that this only seems to affect Presentation screens, but still has a severe impact on mobile with the noted mods.

*Loading a regular stage directly in S1F 1.4.2 causes the game to immediately pause, to which the mod's custom in-game pause menu does not respond to controls or touch, effectively soft-locking it. S2A 1.1.1 does not auto-pause this way, but pausing by pressing Start similarly soft-locks due to unresponsive controls.

The actual bisection has been performed with regards to the Android build specifically, but I tried it also on my (Arch) Linux system. Same exact results there; just replace "touch" with "mouse clicks", and the keyboard and gamepad do and don't work in the same scenarios where the Android pad doesn't work.


Also, FWIW, I've noticed that starting from the affected commit, S1F/S2A skips the mod's Sega logo (both Android & Linux).

And on Linux, S1F outright crashes whenever trying to load a Regular stage:

*** buffer overflow detected ***: terminated
Aborted (core dumped)
MavisCelus19201 commented 1 year ago

Can confirm it still not working even on newer builds when I try to compile, most of the buttons work with the exception of start.

jSQrD-dev commented 1 year ago

This is still the case. As of Dec 17, 2022

Operating System: EndeavourOS
KDE Plasma Version: 5.26.4
KDE Frameworks Version: 5.101.0
Qt Version: 5.15.7
Kernel Version: 6.0.12-zen1-1-zen (64-bit)
Graphics Platform: X11
Processors: 16 × Intel® Core™ i7-10875H CPU @ 2.30GHz
Memory: 15.4 GiB of RAM
Graphics Processor: Mesa Intel® UHD Graphics
Manufacturer: Dell Inc.
Product Name: XPS 17 9700
MavisCelus19201 commented 1 year ago

Tested with the newest build as of January 16th, 2023. Can confirm input is still broken.

OS: Manjaro Linux x86_64 Kernel: 6.1.1-1-MANJARO CPU: Intel i3-7100 (4) @ 3.900GHz [40.0°on] GPU: Intel HD Graphics 630

MegAmi24 commented 1 year ago

Fixed in 93f8767