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
Savestates: QoL improvements #12361
Savestates: QoL improvements #12361
Conversation
elad335
commented
Jul 17, 2022
•
edited
edited
- Fix RAW SPU creation after loading the savestate.
- Fix Ctrl+R reload savestate shortcut bug.
- Fix bugs of vfs::retrieve.
- Do not restart after Ctrl+S for now.
- sys_mutex_unlock saving bugfix.
@@ -301,8 +301,8 @@ class Emulator final | |||
bool Pause(bool freeze_emulation = false); | |||
void Resume(); | |||
void GracefulShutdown(bool allow_autoexit = true, bool async_op = false, bool savestate = false); | |||
void Kill(bool allow_autoexit = true, bool savestate = false); | |||
game_boot_result Restart(bool savestate = false); | |||
std::shared_ptr<utils::serial> Kill(bool allow_autoexit = true, bool savestate = false); |
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.
There doesn't seem to be a reason to return a pointer here.
It's also really confusing as always
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.
This is for future functionality.
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.
The reason that m_ar is no longer used for it is because it can leak without management, here if it's unused it deallocates itself.
Utilities/File.cpp
Outdated
std::string fs::resolve_path(std::string_view path) | ||
{ | ||
std::error_code ec{}; | ||
const auto result = std::filesystem::weakly_canonical(std::filesystem::u8path(path), ec); |
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.
Isn't this one of previously reverted things? Probably need to test it again if it's different now.
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.
The original pr bug has been fixed.