From 2a597da1bc2b974082c8b1fc13c08788b42615af Mon Sep 17 00:00:00 2001
From: Silent <zdanio95@gmail.com>
Date: Thu, 28 Nov 2024 20:41:17 +0100
Subject: [PATCH] III: Backport one more fix from VC to fix the timer
 initialize lingering the audio entities on New Game

Fixes #123
---
 DDraw/dllmain.cpp                 |  4 +++-
 DDraw/versionmeta.props           |  2 +-
 SilentPatch/Utils                 |  2 +-
 SilentPatchIII/SilentPatchIII.cpp | 32 ++++++++++++++++++++++++++++---
 SilentPatchVC/SilentPatchVC.cpp   |  8 ++++----
 SilentPatchVC/versionmeta.props   |  2 +-
 6 files changed, 39 insertions(+), 11 deletions(-)

diff --git a/DDraw/dllmain.cpp b/DDraw/dllmain.cpp
index 9f098f1..e95d1da 100644
--- a/DDraw/dllmain.cpp
+++ b/DDraw/dllmain.cpp
@@ -196,9 +196,11 @@ static bool PatchIAT()
 
 static bool PatchIAT_ByPointers()
 {
+	using namespace Memory::VP;
+
 	pOrgSystemParametersInfoA = SystemParametersInfoA;
 	memcpy( orgCode, pOrgSystemParametersInfoA, sizeof(orgCode) );
-	Memory::VP::InjectHook( pOrgSystemParametersInfoA, SystemParametersInfoA_OverwritingHook, Memory::HookType::Jump );
+	InjectHook( pOrgSystemParametersInfoA, SystemParametersInfoA_OverwritingHook, HookType::Jump );
 	return true;
 }
 
diff --git a/DDraw/versionmeta.props b/DDraw/versionmeta.props
index 44e58c0..7500201 100644
--- a/DDraw/versionmeta.props
+++ b/DDraw/versionmeta.props
@@ -6,7 +6,7 @@
 	<SILENTPATCH_EXT>.dll</SILENTPATCH_EXT>
     <SILENTPATCH_FULL_NAME>SilentPatch DDraw Component</SILENTPATCH_FULL_NAME>
     <SILENTPATCH_REVISION_ID>6</SILENTPATCH_REVISION_ID>
-	<SILENTPATCH_BUILD_ID>0</SILENTPATCH_BUILD_ID>
+	<SILENTPATCH_BUILD_ID>1</SILENTPATCH_BUILD_ID>
     <SILENTPATCH_COPYRIGHT>2014-2024</SILENTPATCH_COPYRIGHT>
   </PropertyGroup>
   <PropertyGroup />
diff --git a/SilentPatch/Utils b/SilentPatch/Utils
index 95e523f..70cab49 160000
--- a/SilentPatch/Utils
+++ b/SilentPatch/Utils
@@ -1 +1 @@
-Subproject commit 95e523fa7494c0353b7ecfab408024a9d7b7e161
+Subproject commit 70cab494523e76d56063b5044c34345c14803c96
diff --git a/SilentPatchIII/SilentPatchIII.cpp b/SilentPatchIII/SilentPatchIII.cpp
index fbb25ac..c29385a 100644
--- a/SilentPatchIII/SilentPatchIII.cpp
+++ b/SilentPatchIII/SilentPatchIII.cpp
@@ -722,7 +722,7 @@ namespace DodoKeyboardControls
 // ============= Resetting stats and variables on New Game =============
 namespace VariableResets
 {
-	static auto TimerInitialise = reinterpret_cast<void(*)()>(hook::get_pattern("83 E4 F8 68 ? ? ? ? E8", -6));
+	static void (*TimerInitialise)();
 
 	using VarVariant = std::variant< bool*, int* >;
 	std::vector<VarVariant> GameVariablesToReset;
@@ -736,8 +736,6 @@ namespace VariableResets
 				}, var );
 		}
 
-		// Functions that should have been called by the game but aren't...
-		TimerInitialise();
 		PurpleNinesGlitchFix();
 	}
 
@@ -757,8 +755,22 @@ namespace VariableResets
 	void GameInitialise(const char* path)
 	{
 		ReInitOurVariables();
+		TimerInitialise();
 		orgGameInitialise(path);
 	}
+
+	static void (__fastcall* DestroyAllGameCreatedEntities)(void* DMAudio);
+
+	template<std::size_t Index>
+	static void (__fastcall* orgService)(void* DMAudio);
+
+	template<std::size_t Index>
+	static void __fastcall Service_AndDestroyEntities(void* DMAudio)
+	{
+		DestroyAllGameCreatedEntities(DMAudio);
+		orgService<Index>(DMAudio);
+	}
+	HOOK_EACH_INIT(Service, orgService, Service_AndDestroyEntities);
 }
 
 
@@ -2354,8 +2366,22 @@ void Patch_III_Common()
 			get_pattern("C6 05 ? ? ? ? ? E8 ? ? ? ? C7 05", 7)
 		};
 
+		TimerInitialise = reinterpret_cast<decltype(TimerInitialise)>(get_pattern("83 E4 F8 68 ? ? ? ? E8", -6));
+
+		// In GTA III, we also need to backport one more fix from VC to avoid issues with looping audio entities:
+		// CMenuManager::DoSettingsBeforeStartingAGame needs to call cDMAudio::DestroyAllGameCreatedEntities
+		DestroyAllGameCreatedEntities = reinterpret_cast<decltype(DestroyAllGameCreatedEntities)>(ReadCallFrom(
+			get_pattern("B9 ? ? ? ? E8 ? ? ? ? 31 DB BD ? ? ? ? 8D 40 00", 5)));
+
+		auto audio_service = pattern("B9 ? ? ? ? E8 ? ? ? ? B9 ? ? ? ? C6 05 ? ? ? ? ? E8").count(2);
+		std::array<void*, 2> audio_service_instances = {
+			audio_service.get(0).get<void>(5),
+			audio_service.get(1).get<void>(5),
+		};
+
 		InterceptCall(game_initialise, orgGameInitialise, GameInitialise);
 		HookEach_ReInitGameObjectVariables(reinit_game_object_variables, InterceptCall);
+		HookEach_Service(audio_service_instances, InterceptCall);
 
 		// Variables to reset
 		GameVariablesToReset.emplace_back(*get_pattern<bool*>("80 3D ? ? ? ? ? 74 2A", 2)); // Free resprays
diff --git a/SilentPatchVC/SilentPatchVC.cpp b/SilentPatchVC/SilentPatchVC.cpp
index c7ba296..003b4ad 100644
--- a/SilentPatchVC/SilentPatchVC.cpp
+++ b/SilentPatchVC/SilentPatchVC.cpp
@@ -1133,7 +1133,7 @@ namespace IsPlayerTargettingCharFix
 // ============= Resetting stats and variables on New Game =============
 namespace VariableResets
 {
-	static auto TimerInitialise = reinterpret_cast<void(*)()>(hook::get_pattern("83 E4 F8 68 ? ? ? ? E8", -6));
+	static void (*TimerInitialise)();
 
 	using VarVariant = std::variant< bool*, int* >;
 	std::vector<VarVariant> GameVariablesToReset;
@@ -1146,9 +1146,6 @@ namespace VariableResets
 				*v = {};
 				}, var );
 		}
-
-		// Functions that should have been called by the game but aren't...
-		TimerInitialise();
 	}
 
 	template<std::size_t Index>
@@ -1167,6 +1164,7 @@ namespace VariableResets
 	void GameInitialise(const char* path)
 	{
 		ReInitOurVariables();
+		TimerInitialise();
 		orgGameInitialise(path);
 	}
 }
@@ -2798,6 +2796,8 @@ void Patch_VC_Common()
 			get_pattern("C6 05 ? ? ? ? ? E8 ? ? ? ? C7 05", 7)
 		};
 
+		TimerInitialise = reinterpret_cast<decltype(TimerInitialise)>(get_pattern("83 E4 F8 68 ? ? ? ? E8", -6));
+
 		InterceptCall(game_initialise, orgGameInitialise, GameInitialise);
 		HookEach_ReInitGameObjectVariables(reinit_game_object_variables, InterceptCall);
 
diff --git a/SilentPatchVC/versionmeta.props b/SilentPatchVC/versionmeta.props
index 66afbb3..28386b1 100644
--- a/SilentPatchVC/versionmeta.props
+++ b/SilentPatchVC/versionmeta.props
@@ -6,7 +6,7 @@
 	<SILENTPATCH_EXT>.asi</SILENTPATCH_EXT>
     <SILENTPATCH_FULL_NAME>SilentPatch for Vice City</SILENTPATCH_FULL_NAME>
     <SILENTPATCH_REVISION_ID>11</SILENTPATCH_REVISION_ID>
-	<SILENTPATCH_BUILD_ID>1</SILENTPATCH_BUILD_ID>
+	<SILENTPATCH_BUILD_ID>2</SILENTPATCH_BUILD_ID>
     <SILENTPATCH_COPYRIGHT>2013-2024</SILENTPATCH_COPYRIGHT>
   </PropertyGroup>
   <PropertyGroup />