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

Gameini updates. #290

Merged
merged 2 commits into from May 3, 2014
Merged

Gameini updates. #290

merged 2 commits into from May 3, 2014

Conversation

Linktothepast
Copy link
Contributor

Updates game inis for Serious Sam: Next Encounter (gc), CAPCOM VS. SNK 2
EO (gc), Baten Kaitos Origins (gc), Baten Kaitos Eternal Wings and the
Lost Ocean (gc), Sonic Adventure 2 Battle (gc), Sonic Riders (gc), Sonic
Adventure DX (gc), Mini Ninjas (wii), The Legend of Zelda Skyward Sword
(wii).
Fixes issues 7146 and 6440.

@neobrain
Copy link
Member

LGTM, even though I'd personally prefer if those per-game options where better commented wrt why specific overrides are necessary.

@Linktothepast
Copy link
Contributor Author

I am thinking of making a pull request just for the explanation of settings. It might take a while to do though.

@neobrain
Copy link
Member

If you're going to do write those explanation, please do it in this PR. There's no rush in merging this PR if it's not actually considered finished, yet.

.. unless you are actually talking about making a PR for the explanation of settings in ALL game settings.

@Linktothepast
Copy link
Contributor Author

It is the second and yes it will take a while, i don't think it is worth this pr to be on hold until then.

@neobrain
Copy link
Member

ack.

Updates game inis for Serious Sam: Next Encounter (gc), CAPCOM VS. SNK 2
EO (gc), Baten Kaitos Origins (gc), Baten Kaitos Eternal Wings and the
Lost Ocean (gc), Sonic Adventure 2 Battle (gc), Sonic Riders (gc), Sonic
Adventure DX (gc), Mini Ninjas (wii), The Legend of Zelda Skyward Sword
(wii).
Fixes issues 7146 and 6440.
For 007: Agent Under Fire (gc), SHADOW THE HEDGEHOG (gc), NFL Street
(gc), NFL Street 2 (gc), NBA STREET V3 (gc).
Fixes issues 7224, 7216 and 7238.
@Linktothepast
Copy link
Contributor Author

Have this pr got forgotten? I ended up making a second commit too.

Parlane added a commit that referenced this pull request May 3, 2014
@Parlane Parlane merged commit 07fbb13 into dolphin-emu:master May 3, 2014
@Linktothepast Linktothepast deleted the gameini_updates branch May 4, 2014 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants