Use direct memory check instead of new variable

Author: samr46

Client/game_sa/CGameSA.cpp

@@ -932,9 +932,6 @@ void CGameSA::SetVehicleBurnExplosionsEnabled(bool isEnabled)
 
 void CGameSA::SetVehicleEngineManualModeEnabled(bool enabled)
 {
-    if (enabled == m_isVehicleEngineManualModeEnabled)
-        return;
-
     if (enabled)
     {
         MemSet((void*)0x64BC03, 0x90, 5);                          // prevent vehicle engine from turning on (driver enter)
@@ -945,8 +942,11 @@ void CGameSA::SetVehicleEngineManualModeEnabled(bool enabled)
         MemCpy((void*)0x64BC03, "\x75\x05\x80\xC9\x10", 5);
         MemCpy((void*)0x6C4EA9, "\x8A\x86\x28\x04", 4);
     }
+}
 
-    m_isVehicleEngineManualModeEnabled = enabled;
+bool CGameSA::IsVehicleEngineManualModeEnabled() const noexcept
+{
+    return *(unsigned char*)0x64BC03 == 0x90;
 }
 
 bool CGameSA::PerformChecks()

Client/game_sa/CGameSA.h

@@ -262,8 +262,8 @@ class CGameSA : public CGame
     bool IsVehicleBurnExplosionsEnabled() const noexcept override { return m_isVehicleBurnExplosionsEnabled; }
     void SetVehicleBurnExplosionsEnabled(bool isEnabled) override;
 
-    bool IsVehicleEngineManualModeEnabled() const noexcept override { return m_isVehicleEngineManualModeEnabled; }
     void SetVehicleEngineManualModeEnabled(bool enabled) override;
+    bool IsVehicleEngineManualModeEnabled() const noexcept override;
 
     unsigned long GetMinuteDuration();
     void          SetMinuteDuration(unsigned long ulTime);
@@ -397,7 +397,6 @@ class CGameSA : public CGame
     bool         m_isExtendedWaterCannonsEnabled{false};
     bool         m_isIgnoreFireStateEnabled{false};
     bool         m_isVehicleBurnExplosionsEnabled{true};
-    bool         m_isVehicleEngineManualModeEnabled{false};
 
     static unsigned int&  ClumpOffset;