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

GameSettings: Enable compatibility patches by default #9317

Merged
merged 3 commits into from
Dec 11, 2020

Conversation

JosJuice
Copy link
Member

This pull request makes it possible for users to disable patches that are enabled by default, and enables patches for game-breaking problems by default.

My criterion for which patches to select was that the patch has to make the game's behavior closer to how the game acts on console. So workarounds for Dolphin not emulating caches accurately are enabled by default, but not things like Twilight Princess's minimap speedhack or patches for disabling memory card checks in Pokémon games.

This is a convenience which I think will be appreciated by people who want to play Resident Evil 2 & 3 Casper's Scare School: Spooky Sports Day.

Copy link
Contributor

@smurf3tte smurf3tte left a comment

Choose a reason for hiding this comment

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

Thanks, I'm eager to start using this! 😄

@JMC47
Copy link
Contributor

JMC47 commented Dec 10, 2020

I don't think it's fair to completely rule out some quality of life patches. I know it's a huge departure from your current criteria of only allowing patches that make games behave more like console, and there's a slippery slope argument, but if you look at it from a usability perspective, there's no reason not to have the save patches on for Pokemon unless they malfunction. How many android users are going to know it's there?

Either way, for the current use? 100% approve.

@JosJuice
Copy link
Member Author

I'm not 100% opposed to extending the criteria, but if we're going to do that, I would like someone to open a separate PR for it so that it doesn't block the merge of this.

How many android users are going to know it's there?

I do plan on adding cheats/patches to the Android GUI at some point :)

Source/Core/Core/ActionReplay.cpp Outdated Show resolved Hide resolved
Source/Core/Core/GeckoCodeConfig.cpp Outdated Show resolved Hide resolved
Source/Core/Core/ActionReplay.cpp Outdated Show resolved Hide resolved
Source/Core/Core/CheatCodes.h Outdated Show resolved Hide resolved
Source/Core/Core/GeckoCodeConfig.cpp Outdated Show resolved Hide resolved
Source/Core/Core/GeckoCodeConfig.cpp Outdated Show resolved Hide resolved
Source/Core/DolphinQt/Config/PatchesWidget.cpp Outdated Show resolved Hide resolved
If we want to enable codes in the default game INIs,
we should have some way for users to disable them.
This commit accomplishes that by adding a *_Disabled
section corresponding to each *_Enabled section.
The previous commit adjusted the code for loading
and this commit adjusts the code for saving.
This enables patches for game-breaking problems by default.

My criterion for which patches to select was that the patch
has to make the game's behavior closer to how the game acts
on console. So workarounds for Dolphin not emulating caches
accurately are enabled by default, but not things like
Twilight Princess's minimap speedhack or patches for
disabling memory card checks in Pokémon games.
@leoetlino leoetlino merged commit 7d9276c into dolphin-emu:master Dec 11, 2020
@JosJuice JosJuice deleted the default-enabled-codes branch December 11, 2020 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants