Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable multitap (multiplayer5) if a button is pressed on a controller other than 1 or 2 #12

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

kyp44
Copy link

@kyp44 kyp44 commented Dec 31, 2015

Code was already in place for multitap support (called multiplayer5 in the code) but it was not enabled. Since there is no way to query retroarch for the number of controllers connected, I implemented a change where the multitap becomes enabled if a button is pressed on any controller other than 1 or 2. Note that I didn't simply enable multitap all the time because I've heard that some 1-2 player games will not work properly with multitap enabled, though I don't know which games these are.

@netux79
Copy link

netux79 commented Dec 31, 2015

I think a better way to implement this would be thru a core option.

@kyp44
Copy link
Author

kyp44 commented Dec 31, 2015

That would probably be a more elegant solution. However, the concern for me (and perhaps with other users) with simply enabling/disabled multitap via a core option is that now you have to go through all your ROMs and enable the option for multitap on ROMs that will use it. This wouldn't be difficult but would be a little time consuming. I'm not familiar with the nature of core options but perhaps the multitap option could be one of three: disabled/enabled/auto where the auto option is how I've implemented it.

@andres-asm
Copy link

not really, core options by default are per-core, unless you create a core options file for a particular game

@kyp44
Copy link
Author

kyp44 commented Dec 31, 2015

I've read somewhere that some games will not work if Multitap is connected/enabled, though I don't know which games these are so haven't been able to test it. If that's true then you wouldn't want MT enabled for all games and you don't want it disabled for games that use it. That's the reason I did it the way I did instead of just always enabling it.

@kyp44
Copy link
Author

kyp44 commented Jan 1, 2016

Also I found such a game. SMW2 Yoshi's Island will not work if multitap is enabled. I found a bug that caused this game to not run properly, which I fixed in my fork.

@andres-asm
Copy link

Core options can be overriden to on a per-game basis too.
I still think that is better than a non-explicit enable but that can be improved on later I guess (auto|on|off) would be great I guess

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants