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

8 player support #326

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

8 player support #326

wants to merge 5 commits into from

Conversation

Yukitty
Copy link

@Yukitty Yukitty commented Oct 7, 2024

Pull Request

General Description

Remake / Update of pull request #175. 🙂

Rebased to current version. Fixed a buffer overflow in an otherwise unused array hiding player 5's HUD due to data misinterpretation. There should be no remaining glitches (it's fully playable and stable at the moment), but I still haven't fully combed through the codebase again double-checking my old work.

Fixes all of the hardcoded array lengths which
are meant to match MAX_PLAYERS.
@Yukitty Yukitty requested a review from DCurrent as a code owner October 7, 2024 06:02
@Yukitty
Copy link
Author

Yukitty commented Oct 7, 2024

If you're still doing input-rework-revival in a separate branch or whatever, I'm still willing to do the work to merge with it myself. I'd like to have input and player slots figured out before adding networking.

Originally posted by @Yukitty in 2019

@Yukitty Yukitty marked this pull request as draft October 8, 2024 00:29
@Yukitty
Copy link
Author

Yukitty commented Oct 8, 2024

It doesn't make sense for the Wii build to show 8 players though, right? Can the console itself even connect with more than 4 controllers at a time?

@Formedras
Copy link

Formedras commented Oct 8, 2024

It doesn't make sense for the Wii build to show 8 players though, right? Can the console itself even connect with more than 4 controllers at a time?

Yes, actually. Four Wii controllers over Bluetooth*, and four wired GameCube controllers. It's not a situation that you'll ever come up across in real use, though, so it might be worth sticking with just 4-player support on that platform.

*To my knowledge, the only controllers for the original Wii that Nintendo released that took a slot were the Wii Remote and the Wii Balance Board. The Pro Controller didn't work on the Wii, but did use a Wii controller slot on the Wii U.

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.

2 participants