dahjah/dolphin
1
0
Fork 0
mirror of https://github.com/dolphin-emu/dolphin.git synced 2025-12-16 04:09:39 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Core/IOS/FS: Clean up some hard to read NAND state saving logic.

Browse Source
This commit is contained in:
Jordan Woyak 2025-10-31 01:46:58 -05:00
parent 249f999c6a
commit 57d7485ea6
1 changed files with 17 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
Source/Core/Core/IOS/FS/HostBackend/FS.cpp
View File

@ -374,11 +374,9 @@ void HostFileSystem::DoState(PointerWrap& p)
handle.host_file.reset(); handle.host_file.reset();
// The format for the next part of the save state is follows: // The format for the next part of the save state is follows:
// 1. bool Movie::WasMovieActiveWhenStateSaved() && // 1. bool is_full_nand_in_state (movie active && temporary wii root)
// WiiRoot::WasWiiRootTemporaryDirectoryWhenStateSaved()
// 2. Contents of the "/tmp" directory recursively. // 2. Contents of the "/tmp" directory recursively.
// 3. u32 size_of_nand_folder_saved_below (or 0, if the root // 3. u32 size_of_nand (or 0, if not is_full_nand_in_state).
// of the NAND folder is not savestated below).
// 4. Contents of the "/" directory recursively (or nothing, if the // 4. Contents of the "/" directory recursively (or nothing, if the
// root of the NAND folder is not save stated). // root of the NAND folder is not save stated).
@ -386,44 +384,42 @@ void HostFileSystem::DoState(PointerWrap& p)
// and when the directory root is temporary (i.e. WiiSession). // and when the directory root is temporary (i.e. WiiSession).
// If a save state is made during a movie recording and is loaded when no movie is active, // If a save state is made during a movie recording and is loaded when no movie is active,
// then a call to p.DoExternal() will be used to skip over reading the contents of the "/" // then a call to p.DoExternal() will be used to skip over reading the contents of the "/"
// directory (it skips over the number of bytes specified by size_of_nand_folder_saved) // directory (it skips over the number of bytes specified by size_of_nand)
auto& movie = Core::System::GetInstance().GetMovie(); auto& movie = Core::System::GetInstance().GetMovie();
bool original_save_state_made_during_movie_recording =
movie.IsMovieActive() && Core::WiiRootIsTemporary();
p.Do(original_save_state_made_during_movie_recording);
u32 temp_val = 0; const bool is_full_nand_wanted = movie.IsMovieActive() && Core::WiiRootIsTemporary();
bool is_full_nand_in_state = is_full_nand_wanted;
p.Do(is_full_nand_in_state);
if (!p.IsReadMode()) if (!p.IsReadMode())
{ {
DoStateWriteOrMeasure(p, "/tmp"); DoStateWriteOrMeasure(p, "/tmp");
u8* previous_position = p.ReserveU32(); u8* const nand_size_ptr = p.ReserveU32();
if (original_save_state_made_during_movie_recording) if (is_full_nand_in_state)
{ {
DoStateWriteOrMeasure(p, "/"); DoStateWriteOrMeasure(p, "/");
if (p.IsWriteMode()) if (p.IsWriteMode())
{ {
u32 size_of_nand = p.GetOffsetFromPreviousPosition(previous_position) - sizeof(u32); const u32 size_of_nand = p.GetOffsetFromPreviousPosition(nand_size_ptr) - sizeof(u32);
memcpy(previous_position, &size_of_nand, sizeof(u32)); memcpy(nand_size_ptr, &size_of_nand, sizeof(size_of_nand));
} }
} }
} }
else // case where we're in read mode. else // case where we're in read mode.
{ {
u32 size_of_nand = 0;
DoStateRead(p, "/tmp"); DoStateRead(p, "/tmp");
if (!movie.IsMovieActive() || !original_save_state_made_during_movie_recording || if (is_full_nand_in_state && is_full_nand_wanted)
!Core::WiiRootIsTemporary() ||
(original_save_state_made_during_movie_recording !=
(movie.IsMovieActive() && Core::WiiRootIsTemporary())))
{ {
(void)p.DoExternal(temp_val); p.Do(size_of_nand);
DoStateRead(p, "/");
} }
else else
{ {
p.Do(temp_val); // Skip over any NAND data without using it.
if (movie.IsMovieActive() && Core::WiiRootIsTemporary()) (void)p.DoExternal(size_of_nand);
DoStateRead(p, "/");
} }
} }