-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
NetPlay: Sync more settings #9571
NetPlay: Sync more settings #9571
Conversation
f70c1ce
to
7397f0f
Compare
Avoiding desyncs is more important than honoring what the user specified on the command line.
Fixes netplay and movie overrides of SYSCONF settings not applying.
Whether the custom RTC setting is enabled shouldn't in itself affect determinism (as long as the actual RTC value is properly synced). Alters the logic added in 4b2906c. I'm not entirely certain that this is correct, but the current code doesn't really make sense to me... If we need to force the RTC bias to 0 when custom RTC is enabled, why don't we need to do it when custom RTC is disabled? The code for getting the host system's current time doesn't contain any special handling for the guest's RTC bias as far as I can tell.
456da59
to
1dfeb73
Compare
Source/Core/Core/BootManager.cpp
Outdated
if (StartUp.bWii) | ||
{ | ||
Core::BackupWiiSettings(); | ||
ConfigLoaders::SaveToSYSCONF(Config::LayerType::Meta); | ||
const bool want_determinism = Movie::IsMovieActive() || NetPlay::IsNetPlayRunning(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to use Core::UpdateWantDeterminism(true) and Core::WantsDeterminism here?
edit: If we use UpdateWantDeterminism, the call would have to be outside the bWii conditional.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added a commit that does this.
Just to make the InitializeWiiRoot call a bit neater.
Fixes certain netplay desyncs, https://bugs.dolphin-emu.org/issues/10328, and a bug where the Wii language stored in a DTM file would be ignored. Prerequisite for PR #9388.