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

IOS/DI: Fake the error 001 read when running DirectoryBlobs or Riivolution-patched games. #10666

Merged
merged 1 commit into from May 22, 2022

Conversation

AdmiralCurtiss
Copy link
Contributor

@JosJuice Is this what you had in mind?

Copy link
Member

@JosJuice JosJuice left a comment

Choose a reason for hiding this comment

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

Not sure if we should do this for all DirectoryBlobs... It's blatantly a hack, but then again, the DirectoryBlobs we create never match "real" ISOs 100% anyway.

(std::holds_alternative<BootParameters::Disc>(boot->parameters) &&
std::get<BootParameters::Disc>(boot->parameters).volume->GetBlobReader().GetBlobType() ==
DiscIO::BlobType::DIRECTORY));
Config::SetCurrent(Config::SESSION_SHOULD_FAKE_ERROR_001, should_fake_error_001);
Copy link
Member

Choose a reason for hiding this comment

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

Ideally this should be implemented in such a way that if the user switches discs, the hack turns on or off to match the new disc.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh right, we do support mid-game disc swapping...

@AdmiralCurtiss
Copy link
Contributor Author

I've enabled it for DirectoryBlobs because I've run into this before while modding -- left a backup copy of an archive in the folder and suddenly it didn't boot anymore. And since that seems like the primary use case for DirectoryBlobs I imagine I'm not the first person to run into that.

@AdmiralCurtiss
Copy link
Contributor Author

@JosJuice So what are we doing with this? IMO this is an acceptable hack considering how DirectoryBlob works.

@JosJuice JosJuice merged commit 5354f69 into dolphin-emu:master May 22, 2022
10 checks passed
@AdmiralCurtiss AdmiralCurtiss deleted the riivolution-error-001 branch May 22, 2022 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants