andwn / cave-story-md

A fan port of Cave Story for the Sega Mega Drive
571 stars 34 forks source link

On Genesis Plus GX, x, y, and z buttons are ignored. #342

Closed crocket closed 2 years ago

crocket commented 3 years ago

Version: 0.7.0

On BlastEm libretro core, x, y, and z buttons are recognized. On Genesis Plus GX libretro core, x, y, and z buttons are ignored.

I compiled the latest commit of https://github.com/libretro/Genesis-Plus-GX

sikthehedgehog commented 3 years ago

What does real hardware do?

This said, just in case you should double check that the GPGX core isn't configured to use a 3-button controller.

crocket commented 3 years ago

I don't have sega genesis.

On cave story, genesis plus gx ignores mode, x, y, and z, but BlastEm recognizes them.

genesis plus gx recognizes x, y, and z on Street Fighter II: Special Champion Edition.

crocket commented 3 years ago

https://github.com/ekeeke/Genesis-Plus-GX/issues/119#issuecomment-266595306 says

That game does not indicate 6-Button support in its ROM header so you will have to force it in Input settings (i.e Joypad AUTO will default to 3-button). You might also need to start the game with 6-button selected since the game might detect it on startup (i.e loading a state which was started with 3-button controller might not work with 6-button controller)

Retroarch's default pad for genesis plus gx is Joypad Auto. Genesis Plus Gx libretro core also has MD Joypad 6 button, MD Joypad 3 button, MD Joypad 6 button + 4-WayPlay, and MD Joypad 3 button + 4-WayPlay.

After manually setting controller Device Type to MD Joypad 6 Button, mode, x, y, and z buttons are recognized.

I think cave-story-md should make genesis plus gx default to 6-button controller by indicating 6-button support in the ROM header.

crocket commented 3 years ago

What is this? It seems that if I press A button, I switch between 3 button mode anf ffwd mode.

Cave Story (en) 0 7 0-211027-093658

sikthehedgehog commented 3 years ago

That's the button to switch weapons.

crocket commented 3 years ago

I think it should be renamed as something less ambiguous like Switch (3btn) / FFwd (6btn). Switch (3btn) and FFwd (btyn) should be in a different color from the color of the separator /. Right now, Switch (3btn) / FFwd looks like Switch between 3btn and FFwd.

At the bottom, an annotation can explain that FFwd means Fast Forward scripted events and text.

andwn commented 2 years ago

Meh.

Screen Shot 2022-01-16 at 0 58 50