all __thiscall & some __cdecl

Author: FileEX

Client/game_sa/CAESoundManagerSA.cpp

@@ -23,7 +23,6 @@ CAESoundManagerSA::CAESoundManagerSA(CAESoundManagerSAInterface* pInterface) : m
 
 void CAESoundManagerSA::CancelSoundsInBankSlot(uint uiGroup, uint uiIndex)
 {
-    using CAESoundManager__CancelSoundsInBankSlot = CAESound*(__thiscall*)(CAESoundManagerSAInterface*, uint, uint);
-    static auto pCancelSoundsInBankSlot = reinterpret_cast<CAESoundManager__CancelSoundsInBankSlot>(FUNC_CAESoundManager__CancelSoundsInBankSlot);
-    pCancelSoundsInBankSlot(m_pInterface, uiGroup, uiIndex);
+    auto args = PrepareSignature(m_pInterface, uiGroup, uiIndex);
+    CallGTAFunction<CAESound*, __THISCALL>(FUNC_CAESoundManager__CancelSoundsInBankSlot, args);
 }

Client/game_sa/CBuildingsPoolSA.cpp

@@ -108,7 +108,8 @@ void CBuildingsPoolSA::RemoveBuilding(CBuilding* pBuilding)
     pGame->GetWorld()->Remove(pInterface, CBuildingPool_Destructor);
 
     // Call virtual destructor
-    ((void*(__thiscall*)(void*, char))pInterface->vtbl->SCALAR_DELETING_DESTRUCTOR)(pInterface, 0);
+    auto args = PrepareSignature(pInterface, 0);
+    CallGTAFunction<void*, __THISCALL>(pInterface->vtbl->SCALAR_DELETING_DESTRUCTOR, args);
 
     // Remove col reference
     auto modelInfo = pGame->GetModelInfo(pBuilding->GetModelIndex());
@@ -137,8 +138,7 @@ void CBuildingsPoolSA::RemoveAllBuildings()
     pGame->GetPlantManager()->RemoveAllPlants();
 
     // Remove all shadows
-    using CStencilShadowObjects_dtorAll = void* (*)();
-    ((CStencilShadowObjects_dtorAll)0x711390)();
+    CallGTAFunction<void*, __CDECL>(0x711390, PrepareSignature());
 
     m_pOriginalBuildingsBackup = std::make_unique<std::array<std::pair<bool, CBuildingSAInterface>, MAX_BUILDINGS>>();
 

Client/game_sa/CCameraSA.cpp

@@ -358,7 +358,8 @@ float CCameraSA::GetCameraRotation()
 RwMatrix* CCameraSA::GetLTM()
 {
     // RwFrameGetLTM
-    return ((RwMatrix*(_cdecl*)(void*))0x7F0990)(GetInterface()->m_pRwCamera->object.object.parent);
+    auto args = PrepareSignature(GetInterface()->m_pRwCamera->object.object.parent);
+    return CallGTAFunction<RwMatrix*, __CDECL>(0x7F0990, args);
 }
 
 CEntity* CCameraSA::GetTargetEntity()
@@ -450,8 +451,8 @@ void CCameraSA::ShakeCamera(float radius, float x, float y, float z) noexcept
     if (radius <= 0.0f)
         return ResetShakeCamera();
 
-    using ShakeCamera_t = void(__thiscall*)(CCameraSAInterface*, float radius, float x, float y, float z);
-    ((ShakeCamera_t)FUNC_ShakeCam)(cameraInterface, radius, x, y, z);
+    auto args = PrepareSignature(cameraInterface, radius, x, y, z);
+    CallGTAFunction<void, __THISCALL>(FUNC_ShakeCam, args);
 }
 
 void CCameraSA::ResetShakeCamera() noexcept

Client/game_sa/CEntitySA.cpp

@@ -25,22 +25,24 @@ extern CGameSA* pGame;
 
 void CEntitySAInterface::TransformFromObjectSpace(CVector& outPosn, CVector const& offset)
 {
-    ((void(__thiscall*)(CEntitySAInterface*, CVector&, CVector const&))0x533560)(this, outPosn, offset);
+    auto args = PrepareSignature(this, &outPosn, &offset);
+    CallGTAFunction<void, __THISCALL>(0x533560, args);
 }
 
 CVector* CEntitySAInterface::GetBoundCentre(CVector* pOutCentre)
 {
-    return ((CVector * (__thiscall*)(CEntitySAInterface*, CVector*))0x534250)(this, pOutCentre);
+    auto args = PrepareSignature(this, pOutCentre);
+    return CallGTAFunction<CVector*, __THISCALL>(0x534250, args);
 }
 
 void CEntitySAInterface::UpdateRW()
 {
-    ((void(__thiscall*)(CEntitySAInterface*))0x446F90)(this);
+    CallGTAFunction<void, __THISCALL>(0x446F90, PrepareSignature(this));
 }
 
 void CEntitySAInterface::UpdateRpHAnim()
 {
-    ((void(__thiscall*)(CEntitySAInterface*))0x532B20)(this);
+    CallGTAFunction<void, __THISCALL>(0x532B20, PrepareSignature(this));
 }
 
 CRect* CEntitySAInterface::GetBoundRect_(CRect* pRect)
@@ -71,6 +73,26 @@ void CEntitySAInterface::StaticSetHooks()
     HookInstall(0x534120, &CEntitySAInterface::GetBoundRect_);
 }
 
+void CEntitySAInterface::ResolveReferences()
+{
+    CallGTAFunction<void*, __THISCALL>(0x571A40, PrepareSignature(this)); 
+}
+
+void CEntitySAInterface::RemoveShadows()
+{
+    CallGTAFunction<void*, __CDECL>(0x711730, PrepareSignature(this));
+}
+
+void CEntitySAInterface::DeleteRwObject()
+{
+    CallGTAFunction<void, __THISCALL>(this->vtbl->DeleteRwObject, PrepareSignature(this));
+}
+
+void CEntitySAInterface::RemoveMatrix()
+{
+    CallGTAFunction<void, __THISCALL>(0x54F3B0, PrepareSignature(this));
+}
+
 CEntitySA::CEntitySA()
 {
     // Set these variables to a constant state
@@ -437,7 +459,8 @@ eEntityStatus CEntitySA::GetEntityStatus()
 RwFrame* CEntitySA::GetFrameFromId(int id)
 {
     // CClumpModelInfo::GetFrameFromId
-    return ((RwFrame*(_cdecl*)(RpClump*, int))0x4C53C0)(m_pInterface->m_pRwObject, id);
+    auto args = PrepareSignature(m_pInterface->m_pRwObject, id);
+    return CallGTAFunction<RwFrame*, __CDECL>(0x4C53C0, args);
 }
 
 RpClump* CEntitySA::GetRpClump()
@@ -448,7 +471,7 @@ RpClump* CEntitySA::GetRpClump()
 RwMatrix* CEntitySA::GetLTMFromId(int id)
 {
     // RwFrameGetLTM
-    return ((RwMatrix*(_cdecl*)(RwFrame*))0x7F0990)(GetFrameFromId(id));
+    return CallGTAFunction<RwMatrix*, __CDECL>(0x7F0990, PrepareSignature(GetFrameFromId(id)));
 }
 
 void CEntitySA::SetAlpha(DWORD dwAlpha)

Client/game_sa/CEntitySA.h

@@ -225,27 +225,15 @@ class CEntitySAInterface
         return -1;
     }
 
-    void ResolveReferences()
-    {
-        using CEntity_ResolveReferences = void*(__thiscall*)(CEntitySAInterface*);
-        ((CEntity_ResolveReferences)0x571A40)(this);
-    };
+    void ResolveReferences();
 
-    void RemoveShadows()
-    {
-        using CStencilShadow_dtorByOwner = void*(__cdecl*)(CEntitySAInterface * pEntity);
-        ((CStencilShadow_dtorByOwner)0x711730)(this);
-    };
+    void RemoveShadows();
 
-    void DeleteRwObject()
-    {
-        using vtbl_DeleteRwObject = void(__thiscall*)(CEntitySAInterface * pEntity);
-        ((vtbl_DeleteRwObject)this->vtbl->DeleteRwObject)(this);
-    };
+    void DeleteRwObject();
 
     bool HasMatrix() const noexcept { return Placeable.matrix != nullptr; }
 
-    void RemoveMatrix() { ((void(__thiscall*)(void*))0x54F3B0)(this); }
+    void RemoveMatrix();
 };
 static_assert(sizeof(CEntitySAInterface) == 0x38, "Invalid size for CEntitySAInterface");
 

Client/game_sa/CFileLoaderSA.cpp

@@ -31,26 +31,26 @@ void CFileLoaderSA::StaticSetHooks()
 CEntitySAInterface* CFileLoaderSA::LoadObjectInstance(SFileObjectInstance* obj)
 {
     // Second argument is model name. It's unused in the function
-    return ((CEntitySAInterface * (__cdecl*)(SFileObjectInstance*, const char*))0x538090)(obj, nullptr);
+    return CallGTAFunction<CEntitySAInterface*, __CDECL>(0x538090, PrepareSignature(obj, static_cast<const char*>(nullptr)));
 }
 
 class CAtomicModelInfo
 {
 public:
-    void CAtomicModelInfo::DeleteRwObject() { ((void(__thiscall*)(CAtomicModelInfo*))(*(void***)this)[8])(this); }
+    void CAtomicModelInfo::DeleteRwObject() { CallGTAFunction<void, __THISCALL>((*(void***)this)[8], PrepareSignature(this)); }
 
-    void CAtomicModelInfo::SetAtomic(RpAtomic* atomic) { ((void(__thiscall*)(CAtomicModelInfo*, RpAtomic*))(*(void***)this)[15])(this, atomic); }
+    void CAtomicModelInfo::SetAtomic(RpAtomic* atomic) { CallGTAFunction<void, __THISCALL>((*(void***)this)[15], PrepareSignature(this, atomic)); }
 };
 
 class CDamagableModelInfo
 {
 public:
-    void CDamagableModelInfo::SetDamagedAtomic(RpAtomic* atomic) { ((void(__thiscall*)(CDamagableModelInfo*, RpAtomic*))0x4C48D0)(this, atomic); }
+    void CDamagableModelInfo::SetDamagedAtomic(RpAtomic* atomic) { CallGTAFunction<void, __THISCALL>(0x4C48D0, PrepareSignature(this, atomic)); }
 };
 
 static char* GetFrameNodeName(RwFrame* frame)
 {
-    return ((char*(__cdecl*)(RwFrame*))0x72FB30)(frame);
+    return CallGTAFunction<char*, __CDECL>(0x72FB30, PrepareSignature(frame));
 }
 
 // Originally there was a possibility for this function to cause buffer overflow
@@ -97,22 +97,22 @@ void GetNameAndDamage(const char* nodeName, char (&outName)[OutBuffSize], bool&
 
 static void CVisibilityPlugins_SetAtomicRenderCallback(RpAtomic* pRpAtomic, RpAtomic* (*renderCB)(RpAtomic*))
 {
-    return ((void(__cdecl*)(RpAtomic*, RpAtomic * (*renderCB)(RpAtomic*)))0x7328A0)(pRpAtomic, renderCB);
+    CallGTAFunction<void, __CDECL>(0x7328A0, PrepareSignature(pRpAtomic, renderCB));
 }
 
 static void CVisibilityPlugins_SetAtomicId(RpAtomic* pRpAtomic, int id)
 {
-    return ((void(__cdecl*)(RpAtomic*, int))0x732230)(pRpAtomic, id);
+    CallGTAFunction<void, __CDECL>(0x732230, PrepareSignature(pRpAtomic, id));
 }
 
 static void CVehicleModelInfo_UseCommonVehicleTexDicationary()
 {
-    ((void(__cdecl*)())0x4C75A0)();
+    CallGTAFunction<void, __CDECL>(0x4C75A0, PrepareSignature());
 }
 
 static void CVehicleModelInfo_StopUsingCommonVehicleTexDicationary()
 {
-    ((void(__cdecl*)())0x4C75C0)();
+    CallGTAFunction<void, __CDECL>(0x4C75C0, PrepareSignature());
 }
 
 static auto          CModelInfo_ms_modelInfoPtrs = (CBaseModelInfoSAInterface**)ARRAY_ModelInfo;
@@ -223,5 +223,5 @@ CEntitySAInterface* CFileLoader_LoadObjectInstance(const char* szLine)
     if (fLenSq > 0.0f && std::fabs(fLenSq - 1.0f) > std::numeric_limits<float>::epsilon())
         inst.rotation /= std::sqrt(fLenSq);
 
-    return ((CEntitySAInterface * (__cdecl*)(SFileObjectInstance*))0x538090)(&inst);
+    return CallGTAFunction<CEntitySAInterface*, __CDECL>(0x538090, PrepareSignature(&inst));
 }

Client/game_sa/CFxManagerSA.cpp

@@ -71,9 +71,8 @@ void CFxManagerSA::OnFxSystemSAInterfaceDestroyed(CFxSystemSAInterface* pFxSyste
 
 CFxSystemBPSAInterface* CFxManagerSA::GetFxSystemBlueprintByName(SString sName)
 {
-    using func_t = CFxSystemBPSAInterface*(__thiscall*)(CFxManagerSAInterface * pInterface, const char* pChars);
-    auto func = reinterpret_cast<func_t>(FUNC_FxManager_c__GetSystemByName);
-    return func(m_pInterface, sName);
+    auto args = PrepareSignature(m_pInterface, static_cast<const char*>(sName));
+    return CallGTAFunction<CFxSystemBPSAInterface*, __THISCALL>(FUNC_FxManager_c__GetSystemByName, args);
 }
 
 bool CFxManagerSA::IsValidFxSystemBlueprintName(SString sName)

Client/game_sa/CFxSA.cpp

@@ -326,6 +326,7 @@ void CFxSA::AddParticle(eFxParticleSystems eFxParticle, const CVector& vecPositi
         newDirection.fZ = (rand() % 10000) * 0.0001f * 4 - 2 + newDirection.fZ;
 
         // Call FxSystem_c::AddParticle
-        ((int(__thiscall*)(FxSystem_c*, const CVector*, const CVector*, float, FxPrtMult_c*, float, float, float, int))FUNC_FXSystem_c_AddParticle)(fxParticleSystem, &vecPosition, &newDirection, 0, &fxPrt, -1.0f, fBrightness, 0, 0);
+        auto args = PrepareSignature(fxParticleSystem, &vecPosition, &newDirection, 0.0f, &fxPrt, -1.0f, fBrightness, 0.0f, 0);
+        CallGTAFunction<int, __THISCALL>(FUNC_FXSystem_c_AddParticle, args);
     }
 }

Client/game_sa/CGameSA.cpp

@@ -675,15 +675,9 @@ void CGameSA::SetWaterCreaturesEnabled(bool isEnabled)
 
     const auto manager = reinterpret_cast<class WaterCreatureManager_c*>(0xC1DF30);
     if (isEnabled)
-    {
-        unsigned char(__thiscall * Init)(WaterCreatureManager_c*) = reinterpret_cast<decltype(Init)>(0x6E3F90);
-        Init(manager);
-    }
+        CallGTAFunction<unsigned char, __THISCALL>(0x6E3F90, PrepareSignature(manager));
     else
-    {
-        void(__thiscall * Exit)(WaterCreatureManager_c*) = reinterpret_cast<decltype(Exit)>(0x6E3FD0);
-        Exit(manager);
-    }
+        CallGTAFunction<void, __THISCALL>(0x6E3FD0, PrepareSignature(manager));
 
     m_areWaterCreaturesEnabled = isEnabled;
 }
@@ -785,17 +779,17 @@ void CGameSA::SetExtendedWaterCannonsEnabled(bool isEnabled)
     {
         char* currentCannon = (char*)currentACannons + i * SIZE_CWaterCannon;
 
-        ((void(__thiscall*)(int, void*, bool))FUNC_CAESoundManager_CancelSoundsOwnedByAudioEntity)(STRUCT_CAESoundManager, currentCannon + NUM_CWaterCannon_Audio_Offset, true); // CAESoundManager::CancelSoundsOwnedByAudioEntity to prevent random crashes from CAESound::UpdateParameters
-        ((void(__thiscall*)(void*))FUNC_CWaterCannon_Destructor)(currentCannon); // CWaterCannon::~CWaterCannon
+        CallGTAFunction<void, __THISCALL>(FUNC_CAESoundManager_CancelSoundsOwnedByAudioEntity, PrepareSignature(STRUCT_CAESoundManager, static_cast<void*>(currentCannon + NUM_CWaterCannon_Audio_Offset), true)); // CAESoundManager::CancelSoundsOwnedByAudioEntity to prevent random crashes from CAESound::UpdateParameters
+        CallGTAFunction<void, __THISCALL>(FUNC_CWaterCannon_Destructor, PrepareSignature(static_cast<void*>(currentCannon))); // CWaterCannon::~CWaterCannon
     }
 
     // Call CWaterCannon constructor & CWaterCannon::Init
     for (int i = 0; i < newLimit; ++i)
     {
-        char* currentCannon = (char*)aCannons + i * SIZE_CWaterCannon;
+        void* currentCannon = (char*)aCannons + i * SIZE_CWaterCannon;
 
-        ((void(__thiscall*)(void*))FUNC_CWaterCannon_Constructor)(currentCannon); // CWaterCannon::CWaterCannon
-        ((void(__thiscall*)(void*))FUNC_CWaterCannon_Init)(currentCannon); // CWaterCannon::Init
+        CallGTAFunction<void, __THISCALL>(FUNC_CWaterCannon_Constructor, PrepareSignature(currentCannon)); // CWaterCannon::CWaterCannon
+        CallGTAFunction<void, __THISCALL>(FUNC_CWaterCannon_Init, PrepareSignature(currentCannon)); // CWaterCannon::Init
     }
 
     // Patch references to array
@@ -831,7 +825,7 @@ void CGameSA::SetExtendedWaterCannonsEnabled(bool isEnabled)
     MemPut<BYTE>(0x856BF6, newLimit);
 
     // Free previous allocated memory
-    if (!isEnabled && currentACannons != nullptr)
+    if (!isEnabled && currentACannons)
         free(currentACannons);
 
     m_isExtendedWaterCannonsEnabled = isEnabled;

Client/game_sa/CMatrixSA.cpp

@@ -3,30 +3,34 @@
 
 CMatrixSAInterface::CMatrixSAInterface(CMatrixSAInterface const& matrix)
 {
-    ((void(__thiscall*)(CMatrixSAInterface*, CMatrixSAInterface const&))0x59BCF0)(this, matrix);
+    auto args = PrepareSignature(this, &matrix);
+    CallGTAFunction<void, __THISCALL>(CMatrix_Constructor, args);
 }
 
 CMatrixSAInterface::CMatrixSAInterface(RwMatrix* matrix, bool temporary)
 {
-    ((void(__thiscall*)(CMatrixSAInterface*, RwMatrix*, bool))0x59C050)(this, matrix, temporary);
+    auto args = PrepareSignature(this, matrix, temporary);
+    CallGTAFunction<void, __THISCALL>(CMatrix_Constructor2, args);
 }
 
 // destructor detaches matrix if attached
 CMatrixSAInterface::~CMatrixSAInterface()
 {
-    ((void(__thiscall*)(CMatrixSAInterface*))0x59ACD0)(this);
+    CallGTAFunction<void, __THISCALL>(CMatrix_Destructor, PrepareSignature(this));
 }
 
 void CMatrixSAInterface::ConvertToEulerAngles(float& x, float& y, float& z, std::int32_t flags)
 {
-    ((void(__thiscall*)(CMatrixSAInterface*, float&, float&, float&, std::int32_t))0x59A840)(this, x, y, z, flags);
+    auto args = PrepareSignature(this, &x, &y, &z, flags);
+    CallGTAFunction<void, __THISCALL>(FUNC_CMatrix_ConvertToEulerAngles, args);
 }
 void CMatrixSAInterface::ConvertFromEulerAngles(float x, float y, float z, std::int32_t flags)
 {
-    ((void(__thiscall*)(CMatrixSAInterface*, float, float, float, std::int32_t))0x59AA40)(this, x, y, z, flags);
+    auto args = PrepareSignature(this, x, y, z, flags);
+    CallGTAFunction<void, __THISCALL>(FUNC_CMatrix_ConvertFromEulerAngles, args);
 }
 
 void CMatrixSAInterface::UpdateRW()
 {
-    ((void(__thiscall*)(CMatrixSAInterface*))0x59BBB0)(this);
+    CallGTAFunction<void, __THISCALL>(FUNC_CMatrix_UpdateRW, PrepareSignature(this));
 }