Load save-state:

. fix for II/II+ when VidHD's SHR is being updated (init memVidHD ptr). . refactor to consolidate all Mem* related initialisation.
jvernet · Dec 23, 2021 · a727db7 · a727db7
1 parent 0f2d4b1
commit a727db7
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 14 deletions.
diff --git a/source/Memory.cpp b/source/Memory.cpp
@@ -1731,22 +1731,45 @@ void MemInitializeIO(void)
 
 // Called by:
 // . Snapshot_LoadState_v2()
-void MemInitializeCardSlotAndExpansionRomFromSnapshot(void)
+void MemInitializeFromSnapshot(void)
 {
+	MemInitializeROM();
+	MemInitializeCustomROM();
+	MemInitializeCustomF8ROM();
+	MemInitializeIO();
+
+	//
+	// Card and Expansion ROM
+	//
+
 	// Remove all the cards' ROMs at $Csnn if internal ROM is enabled
 	if (IsAppleIIeOrAbove(GetApple2Type()) && SW_INTCXROM)
 		IoHandlerCardsOut();
 
 	// Potentially init a card's expansion ROM
 	const UINT uSlot = g_uPeripheralRomSlot;
 
-	if (ExpansionRom[uSlot] == NULL)
-		return;
+	if (ExpansionRom[uSlot] != NULL)
+	{
+		_ASSERT(g_eExpansionRomType == eExpRomPeripheral);
 
-	_ASSERT(g_eExpansionRomType == eExpRomPeripheral);
+		memcpy(pCxRomPeripheral + 0x800, ExpansionRom[uSlot], FIRMWARE_EXPANSION_SIZE);
+		// NB. Copied to /mem/ by UpdatePaging(TRUE)
+	}
+
+	MemUpdatePaging(TRUE);
+
+	//
+	// VidHD
+	//
 
-	memcpy(pCxRomPeripheral+0x800, ExpansionRom[uSlot], FIRMWARE_EXPANSION_SIZE);
-	// NB. Copied to /mem/ by UpdatePaging(TRUE)
+	memVidHD = NULL;
+
+	if (IsApple2PlusOrClone(GetApple2Type()) && (GetCardMgr().QuerySlot(SLOT3) == CT_VidHD))
+	{
+		VidHDCard* vidHD = dynamic_cast<VidHDCard*>(GetCardMgr().GetObj(SLOT3));
+		memVidHD = vidHD->IsWriteAux() ? memaux : NULL;
+	}
 }
 
 inline DWORD getRandomTime()
@@ -1778,6 +1801,8 @@ void MemReset()
 
 	memset(memdirty, 0, 0x100);
 
+	memVidHD = NULL;
+
 	//
 
 	int iByte;

diff --git a/source/Memory.h b/source/Memory.h
@@ -84,7 +84,7 @@ void    MemInitializeROM(void);
 void    MemInitializeCustomROM(void);
 void    MemInitializeCustomF8ROM(void);
 void    MemInitializeIO(void);
-void    MemInitializeCardSlotAndExpansionRomFromSnapshot(void);
+void    MemInitializeFromSnapshot(void);
 BYTE    MemReadFloatingBus(const ULONG uExecutedCycles);
 BYTE    MemReadFloatingBus(const BYTE highbit, const ULONG uExecutedCycles);
 void    MemReset ();

diff --git a/source/SaveState.cpp b/source/SaveState.cpp
@@ -499,13 +499,7 @@ static void Snapshot_LoadState_v2(void)
 		const CConfigNeedingRestart configNew = CConfigNeedingRestart::Create();
 		GetPropertySheet().ApplyNewConfigFromSnapshot(configNew);	// Saves new state to Registry (not slot/cards though)
 
-		MemInitializeROM();
-		MemInitializeCustomROM();
-		MemInitializeCustomF8ROM();
-		MemInitializeIO();
-		MemInitializeCardSlotAndExpansionRomFromSnapshot();
-
-		MemUpdatePaging(TRUE);
+		MemInitializeFromSnapshot();
 
 		DebugReset();
 		if (g_nAppMode == MODE_DEBUG)