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

Famicom/NES - Added SNES-2-NES Controller Adaptor support. #1340

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Kalixtan
Copy link
Contributor

@Kalixtan Kalixtan commented Dec 9, 2023

Dout anyone will use it as only a very few bits of homebrew use the extra inputs. But the NES-2-SNES cable is added, all keys work.
If anyone wants i can add other controllers other than the "GamePad" one useing this adapter,but yet to see any software that uses them.

The other way is possible but doubtful its worth emulating as it should just work identically to the SFC controller with buttons missing.

Dout anyone will use it as only a very few bits of homebrew use the extra inputs.
But the NES-2-SNES cable is added, all keys work.
If anyone wants i can add other controllers but dont think any software supports other add-ons useing this basic cable.
@@ -0,0 +1,94 @@
SFC_Gamepad::SFC_Gamepad(Node::Port parent) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: We don't usually use underscores in class names; consider SuperFamicomController, as SFCController looks awkward.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make whatever changes you want, dont have time to deal with this right now, personal things are happening in my life and its likey 5 seconds for you anyways....

Same with the other pull request for those Coleco-Vision Mappers.

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

Successfully merging this pull request may close these issues.

2 participants