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

Disable Wii Save Options When Emulation Is Running #12093

Merged
merged 1 commit into from Aug 12, 2023

Conversation

Hark64
Copy link
Contributor

@Hark64 Hark64 commented Aug 10, 2023

This PR fixes an issue with export wii save files that is detailed here.

Trying to export a wii save while emulation is running creates some error messages and then appears to export the wii save normally but these are options that, in the previous UI of dolphin, were disabled during emulation.

@mbc07
Copy link
Contributor

mbc07 commented Aug 11, 2023

Exporting saves goes through IOS emulation code so I can see how the operation can fail when a game is running, but why are you also disabling "Open Wii Save Folder"? The user can still access that folder directly while a game is running via other means, so I'm not sure what exactly this would prevent/avoid...

@Hark64
Copy link
Contributor Author

Hark64 commented Aug 11, 2023

Oh sorry, I thought it would have been better to make it work like the WX ui, I can remove that.

I think that this change should be kept though because it would be better to make it harder for a user to touch a wii save during emulation than to do nothing at all.
PR #5185, the PR that added those disabled options for the WX ui also expresses some similar sentiments.

Copy link
Contributor

@mbc07 mbc07 left a comment

Choose a reason for hiding this comment

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

Okay, fair enough.

You should squash the commits and fix the lint issues, though...

@AdmiralCurtiss AdmiralCurtiss merged commit bc47a28 into dolphin-emu:master Aug 12, 2023
11 checks passed
@Hark64 Hark64 deleted the exportWiiSaveErrorFix branch August 15, 2023 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants