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

GBA: Add import and export save options to context menu. #10240

Merged
merged 1 commit into from Nov 21, 2021

Conversation

AdmiralCurtiss
Copy link
Contributor

Since users seem to be very confused about where save files should go for the emulator to pick them up.

@@ -229,6 +229,36 @@ void GBAWidget::DoState(bool export_state)
false);
}

void GBAWidget::ImportExportSave(bool export_save)
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't personally have a strong opinion about this but I know some people prefer an enumeration over a boolean

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just copied the existing DoState() more or less, but yeah, probably.

Copy link
Contributor

@iwubcode iwubcode left a comment

Choose a reason for hiding this comment

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

Untested, LGTM

@JMC47
Copy link
Contributor

JMC47 commented Nov 21, 2021

This is probably an improvement. Does this handle that each GBA uses a different save?

@AdmiralCurtiss
Copy link
Contributor Author

It does handle that correctly, yes.

@JMC47
Copy link
Contributor

JMC47 commented Nov 21, 2021

Tested with all 4 GBAs

@JMC47 JMC47 merged commit 37247a5 into dolphin-emu:master Nov 21, 2021
10 checks passed
@AdmiralCurtiss AdmiralCurtiss deleted the mgba-save-import-export branch November 21, 2021 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants