Fix hooks passing references to local variables to registers

Author: Starkku

src/Ext/EBolt/Hooks.cpp

@@ -155,12 +155,13 @@ DEFINE_HOOK(0x4C285D, EBolt_DrawAll_BurstIndex, 0x5)
 
 	GET(TechnoClass*, pTechno, ECX);
 	GET_STACK(EBolt*, pThis, STACK_OFFSET(0x34, -0x24));
+	LEA_STACK(CoordStruct*, pCoords, STACK_OFFSET(0x34, -0xC));
 
 	int burstIndex = pTechno->CurrentBurstIndex;
 	pTechno->CurrentBurstIndex = EBoltExt::ExtMap.Find(pThis)->BurstIndex;
-	auto const fireCoords = pTechno->GetFLH(pThis->WeaponSlot, CoordStruct::Empty);
+	pTechno->GetFLH(pCoords, pThis->WeaponSlot, CoordStruct::Empty);
 	pTechno->CurrentBurstIndex = burstIndex;
-	R->EAX(&fireCoords);
+	R->EAX(pCoords);
 
 	return SkipGameCode;
 }

src/Misc/Hooks.BugFixes.cpp

@@ -567,14 +567,12 @@ namespace FetchBomb
 DEFINE_HOOK(0x438771, BombClass_Detonate_SetContext, 0x6)
 {
 	GET(BombClass*, pThis, ESI);
+	GET(CoordStruct*, pCoords, EDX);
 
 	FetchBomb::pThisBomb = pThis;
 
 	if (RulesExt::Global()->IvanBombAttachToCenter)
-	{
-		CoordStruct coords = pThis->Target->GetCenterCoords();
-		R->EDX(&coords);
-	}
+		*pCoords = pThis->Target->GetCenterCoords();
 
 	return 0;
 }
@@ -624,12 +622,10 @@ DEFINE_FUNCTION_JUMP(CALL, 0x4387A3, _BombClass_Detonate_DamageArea);
 DEFINE_HOOK(0x6F5201, TechnoClass_DrawExtras_IvanBombImage, 0x6)
 {
 	GET(TechnoClass*, pThis, EBP);
+	GET(CoordStruct*, pCoords, EAX);
 
 	if (RulesExt::Global()->IvanBombAttachToCenter)
-	{
-		auto coords = pThis->GetCenterCoords();
-		R->EAX(&coords);
-	}
+		*pCoords = pThis->GetCenterCoords();
 
 	return 0;
 }
@@ -653,12 +649,10 @@ DEFINE_HOOK(0x43D874, BuildingClass_Draw_BuildupBibShape, 0x6)
 DEFINE_HOOK(0x70BCE6, TechnoClass_GetTargetCoords_BuildingFix, 0x6)
 {
 	GET(TechnoClass*, pThis, ESI);
+	GET(CoordStruct*, pCoords, EAX);
 
 	if (const auto pBuilding = abstract_cast<BuildingClass*>(pThis->Target))
-	{
-		const auto coords = pBuilding->GetTargetCoords();
-		R->EAX(&coords);
-	}
+		*pCoords = pBuilding->GetTargetCoords();
 
 	return 0;
 }