Small fixes

Author: TheNormalnij

Client/game_sa/CBuildingsPoolSA.h

@@ -43,8 +43,8 @@ class CBuildingsPoolSA : public CBuildingsPool
     SVectorPoolData<CBuildingSA> m_buildingPool{MAX_BUILDINGS};
     CPoolSAInterface<CBuildingSAInterface>**           m_ppBuildingPoolInterface;
 
-    typedef std::uint8_t building_buffer_t[sizeof(CBuildingSAInterface)];
-    typedef std::array<std::pair<bool, building_buffer_t>, MAX_BUILDINGS> backup_array_t;
+    using building_buffer_t = std::uint8_t[sizeof(CBuildingSAInterface)];
+    using backup_array_t = std::array<std::pair<bool, building_buffer_t>, MAX_BUILDINGS>;
 
     std::unique_ptr<backup_array_t> m_pOriginalBuildingsBackup;
 };

Client/game_sa/CEntitySA.h

@@ -216,8 +216,6 @@ class CEntitySAInterface : public CPlaceableSAInterface
         using CStencilShadow_dtorByOwner = void*(__cdecl*)(CEntitySAInterface * pEntity);
         ((CStencilShadow_dtorByOwner)0x711730)(this);
     };
-    
-    bool IsUsesEntityDeleteRwObject() const { return *reinterpret_cast<void**>(*reinterpret_cast<std::uint32_t const*>(this) + 0x20) == (void*)0x00534030; };
 };
 static_assert(sizeof(CEntitySAInterface) == 0x38, "Invalid size for CEntitySAInterface");
 

Client/game_sa/CModelInfoSA.cpp

@@ -985,14 +985,15 @@ void CModelInfoSA::StaticFlushPendingRestreamIPL()
             CEntitySAInterface* pEntity = (CEntitySAInterface*)pSectorEntry[0];
 
             // Possible bug - pEntity seems to be invalid here occasionally
-            if (!pEntity->IsUsesEntityDeleteRwObject())
+            constexpr auto CEntity_DeleteRwObject_VTBL_OFFSET = 8;
+            if (static_cast<std::size_t*>(pEntity->GetVTBL())[CEntity_DeleteRwObject_VTBL_OFFSET] != 0x00534030)
             {
                 // Log info
                 OutputDebugString(SString("Entity 0x%08x (with model %d) at ARRAY_StreamSectors[%d,%d] is invalid\n", pEntity, pEntity->m_nModelIndex,
                                           i / 2 % NUM_StreamSectorRows, i / 2 / NUM_StreamSectorCols));
                 // Assert in debug
                 #if MTA_DEBUG
-                assert(pEntity->IsUsesEntityDeleteRwObject());
+                assert(static_cast<std::size_t*>(pEntity->GetVTBL())[CEntity_DeleteRwObject_VTBL_OFFSET] != 0x00534030);
                 #endif
                 pSectorEntry = (DWORD*)pSectorEntry[1];
                 continue;

Client/game_sa/CPlaceableSA.h

@@ -22,8 +22,10 @@ class CPlaceableSAInterface
 public:
     virtual void* Destructor(bool free) = 0;
 
-    bool IsPlaceableVTBL() const { return *reinterpret_cast<const void* const*>(this) == (void*)0x863C40; }
-    bool HasVTBL() const { return *reinterpret_cast<const void* const*>(this) != nullptr; }
+    void* GetVTBL() const { return *reinterpret_cast<void* const*>(this); }
+    bool  HasVTBL() const { return GetVTBL() != nullptr; }
+    bool  IsPlaceableVTBL() const { return GetVTBL() == (void*)0x863C40; }
+
     bool HasMatrix() const noexcept { return matrix != nullptr; }
     void RemoveMatrix() { ((void(__thiscall*)(void*))0x54F3B0)(this); }
 

Client/game_sa/CProjectileSA.h

@@ -17,7 +17,7 @@
 class CProjectileSAInterface : public CObjectSAInterface            // entirely inherited from CObject
 {
 public:
-    bool IsProjectableVTBL() const { return *reinterpret_cast<const std::uint32_t*>(this) == 0x867030; };
+    bool IsProjectableVTBL() const { return GetVTBL() == (void*)0x867030; };
 };
 
 class CProjectileSA : public virtual CProjectile, public virtual CObjectSA