Merge ff4112c3d2 into 202101e4f7

SilentPatchVC: Explicitly define _USE_MATH_DEFINES
Not doing that makes M_PI disappear (possibly) due to rwcore.h secretly including a math header (which breaks Maths.h on MinGW GCC)
2025-01-01 16:53:01 +05:00 · 2024-11-07 12:43:33 +02:00 · 2024-11-07 10:43:28 +02:00 · 2024-11-07 10:43:28 +02:00 · 2024-11-07 10:43:28 +02:00 · 2024-11-07 10:43:28 +02:00
8 changed files with 25 additions and 33 deletions
--- a/SilentPatch/Common.h
+++ b/SilentPatch/Common.h
@ -2,13 +2,6 @@

 #include <cstdint>

-// Workaround for old MSVC inline function bugs
-#if defined(_MSC_VER) && _MSC_VER < 1930
-#define STATIC_INLINE static
-#else
-#define STATIC_INLINE static inline
-#endif
-
 #define RwEngineInstance (*rwengine)

 extern "C" void** rwengine;
--- a/SilentPatchIII/SilentPatchIII.cpp
+++ b/SilentPatchIII/SilentPatchIII.cpp
@ -1254,7 +1254,7 @@ namespace SlidingTextsScalingFixes
 		static inline float** pHorShadowValue;

 		template<std::size_t Index>
-		STATIC_INLINE void (*orgPrintString)(float,float,const wchar_t*);
+		static inline void (*orgPrintString)(float,float,const wchar_t*);

 		template<std::size_t Index>
 		static void PrintString_Slide(float fX, float fY, const wchar_t* pText)
@ -1281,7 +1281,7 @@ namespace SlidingTextsScalingFixes
 		static inline bool bSlidingEnabled = false;

 		template<std::size_t Index>
-		STATIC_INLINE void (*orgPrintString)(float,float,const wchar_t*);
+		static inline void (*orgPrintString)(float,float,const wchar_t*);

 		template<std::size_t Index>
 		static void PrintString_Slide(float fX, float fY, const wchar_t* pText)
--- a/SilentPatchSA/AudioHardwareSA.h
+++ b/SilentPatchSA/AudioHardwareSA.h
@ -2,7 +2,7 @@
 #define __AUDIOHARDWARE

 // IStream
-#include <objidl.h>
+#include <Objidl.h>

 enum eDecoderType
 {
@ -205,4 +205,4 @@ public:
 static_assert(sizeof(CAEDataStreamOld) == 0x28, "Wrong size: CAEDataStreamOld");
 static_assert(sizeof(CAEDataStreamNew) == 0x2C, "Wrong size: CAEDataStreamNew");

-#endif
+#endif
--- a/SilentPatchSA/FLACDecoderSA.h
+++ b/SilentPatchSA/FLACDecoderSA.h
@ -3,8 +3,8 @@
 #include "AudioHardwareSA.h"

 // libflac
-#include "FLAC/stream_decoder.h"
-#include "FLAC/metadata.h"
+#include "FLAC\stream_decoder.h"
+#include "FLAC\metadata.h"

 class CAEFLACDecoder final : public CAEStreamingDecoder
 {
@ -45,4 +45,4 @@ public:
 	{ return m_streamMeta->data.stream_info.sample_rate; }
 	virtual uint32_t		GetStreamID() const override
 	{ return GetStream()->GetID(); }
-};
+};
--- a/SilentPatchSA/GeneralSA.h
+++ b/SilentPatchSA/GeneralSA.h
@ -2,7 +2,6 @@
 #define __GENERAL

 #include <stdint.h>
-#include "Common.h"
 #include "TheFLAUtils.h"

 class CSimpleTransform
@ -571,4 +570,4 @@ static_assert(sizeof(CPhysical) == 0x138, "Wrong size: CPhysical");
 static_assert(sizeof(CObject) == 0x17C, "Wrong size: CObject");
 static_assert(sizeof(CEscalator) == 0x150, "Wrong size: CEscalator");

-#endif
+#endif
--- a/SilentPatchSA/SilentPatchSA.cpp
+++ b/SilentPatchSA/SilentPatchSA.cpp
@ -3,9 +3,9 @@
 #include <algorithm>
 #include <array>
 #include <d3d9.h>
-#include <shlwapi.h>
-#include <shlobj.h>
-#include <shellapi.h>
+#include <Shlwapi.h>
+#include <ShlObj.h>
+#include <ShellAPI.h>
 #include <cinttypes>

 #include "ScriptSA.h"
@ -2851,7 +2851,7 @@ namespace CreditsScalingFixes
 	static const unsigned int FIXED_RES_HEIGHT_SCALE = 448;

 	template<std::size_t Index>
-	STATIC_INLINE void (*orgPrintString)(float,float,const wchar_t*);
+	static void (*orgPrintString)(float,float,const wchar_t*);

 	template<std::size_t Index>
 	static void PrintString_ScaleY(float fX, float fY, const wchar_t* pText)
@ -2888,7 +2888,7 @@ namespace SlidingTextsScalingFixes
 		static inline bool bSlidingEnabled = false;

 		template<std::size_t Index>
-		STATIC_INLINE void (*orgPrintString)(float,float,const wchar_t*);
+		static void (*orgPrintString)(float,float,const wchar_t*);

 		template<std::size_t Index>
 		static void PrintString_Slide(float fX, float fY, const wchar_t* pText)
@ -2898,7 +2898,7 @@ namespace SlidingTextsScalingFixes
 		}

 		template<std::size_t Index>
-		STATIC_INLINE void (*orgSetRightJustifyWrap)(float wrap);
+		static void (*orgSetRightJustifyWrap)(float wrap);

 		template<std::size_t Index>
 		static void SetRightJustifyWrap_Slide(float wrap)
@ -2915,7 +2915,7 @@ namespace SlidingTextsScalingFixes
 		static inline bool bSlidingEnabled = false;

 		template<std::size_t Index>
-		STATIC_INLINE void (*orgPrintString)(float,float,const wchar_t*);
+		static void (*orgPrintString)(float,float,const wchar_t*);

 		template<std::size_t Index>
 		static void PrintString_Slide(float fX, float fY, const wchar_t* pText)
@ -3961,7 +3961,7 @@ DarkVehiclesFix4_MakeItDark:
 }
 // 1.0 ONLY ENDS HERE

-NOBUFFERCHECKS static int _Timers_ftol_internal( double timer, double& remainder )
+__declspec(safebuffers) static int _Timers_ftol_internal( double timer, double& remainder )
 {
 	double integral;
 	remainder = modf( timer + remainder, &integral );
--- a/SilentPatchSA/VehicleSA.h
+++ b/SilentPatchSA/VehicleSA.h
@ -302,7 +302,7 @@ public:

 private:
 	template<std::size_t Index>
-	STATIC_INLINE void (CVehicle::*orgDoHeadLightBeam)(int type, CMatrix& m, bool right);
+	static void (CVehicle::*orgDoHeadLightBeam)(int type, CMatrix& m, bool right);

 	template<std::size_t Index>
 	void DoHeadLightBeam_LightBeamFixSaveObj(int type, CMatrix& m, bool right)
@ -337,7 +337,7 @@ public:

 public:
 	template<std::size_t Index>
-	STATIC_INLINE void (CAutomobile::*orgAutomobilePreRender)();
+	static void (CAutomobile::*orgAutomobilePreRender)();

 	template<std::size_t Index>
 	void		PreRender_SilentPatch()
@ -352,7 +352,7 @@ public:
 	void HideDestroyedWheels_SilentPatch(void (CAutomobile::*spawnFlyingComponentCB)(int, unsigned int), int nodeID, unsigned int modelID);

 	template<std::size_t Index>
-	STATIC_INLINE void (CAutomobile::*orgSpawnFlyingComponent)(int, unsigned int);
+	static void (CAutomobile::*orgSpawnFlyingComponent)(int, unsigned int);

 	template<std::size_t Index>
 	void		SpawnFlyingComponent_HideWheels(int nodeID, unsigned int modelID)
@ -448,7 +448,7 @@ private:

 private:
 	template<std::size_t Index>
-	STATIC_INLINE CVehicle* (CStoredCar::*orgRestoreCar)();
+	static CVehicle* (CStoredCar::*orgRestoreCar)();
 	
 	template<std::size_t Index>
 	CVehicle* RestoreCar_SilentPatch()
@ -477,4 +477,4 @@ static_assert(sizeof(CVehicle) == 0x5A0, "Wrong size: CVehicle");
 static_assert(sizeof(CAutomobile) == 0x988, "Wrong size: CAutomobile");
 static_assert(sizeof(CStoredCar) == 0x40, "Wrong size: CStoredCar");

-#endif
+#endif
--- a/SilentPatchVC/SilentPatchVC.cpp
+++ b/SilentPatchVC/SilentPatchVC.cpp
@ -356,7 +356,7 @@ namespace RadarTraceOutlineFixes
 		}

 		template<std::size_t Index>
-		STATIC_INLINE void (*orgShowRadarTraceWithHeight)(float, float, unsigned int, unsigned char, unsigned char, unsigned char, unsigned char, unsigned char);
+		static inline void (*orgShowRadarTraceWithHeight)(float, float, unsigned int, unsigned char, unsigned char, unsigned char, unsigned char, unsigned char);

 		template<std::size_t Index>
 		static void ShowRadarTraceWithHeight_RecalculatePositions(float a1, float a2, unsigned int a3, unsigned char a4, unsigned char a5, unsigned char a6, unsigned char a7, unsigned char a8)
@ -456,7 +456,7 @@ namespace SlidingTextsScalingFixes
 		static inline bool bSlidingEnabled = false;

 		template<std::size_t Index>
-		STATIC_INLINE void (*orgPrintString)(float,float,const wchar_t*);
+		static inline void (*orgPrintString)(float,float,const wchar_t*);

 		template<std::size_t Index>
 		static void PrintString_Slide(float fX, float fY, const wchar_t* pText)
@ -466,7 +466,7 @@ namespace SlidingTextsScalingFixes
 		}

 		template<std::size_t Index>
-		STATIC_INLINE void (*orgSetRightJustifyWrap)(float wrap);
+		static inline void (*orgSetRightJustifyWrap)(float wrap);

 		template<std::size_t Index>
 		static void SetRightJustifyWrap_Slide(float wrap)
@ -483,7 +483,7 @@ namespace SlidingTextsScalingFixes
 		static inline bool bSlidingEnabled = false;

 		template<std::size_t Index>
-		STATIC_INLINE void (*orgPrintString)(float,float,const wchar_t*);
+		static inline void (*orgPrintString)(float,float,const wchar_t*);

 		template<std::size_t Index>
 		static void PrintString_Slide(float fX, float fY, const wchar_t* pText)
Author	SHA1	Message	Date
Echo J.	8a75bc73f2	Merge `ff4112c3d2` into `202101e4f7`	2024-11-07 12:43:33 +02:00
Echo J	ff4112c3d2	SilentPatchVC: Explicitly define _USE_MATH_DEFINES Not doing that makes M_PI disappear (possibly) due to rwcore.h secretly including a math header (which breaks Maths.h on MinGW GCC)	2024-11-07 10:43:28 +02:00
Echo J	9d260b493c	SilentPatchVC: Add GCC/Clang-specific inline assembly statements This also includes a small wrapper to call a C++ function from inline ASM (From III: This change also matches the MSVC statements more closely to the GCC/Clang ones)	2024-11-07 10:43:28 +02:00
Echo J	66c3c364e8	SilentPatchIII: Add GCC/Clang-specific inline assembly statements This also matches the MSVC statements more closely to the GCC/Clang ones	2024-11-07 10:43:28 +02:00
Echo J	4d319e08ee	SilentPatchIII: Add a missing functional header include MinGW GCC doesn't implicitly include it either	2024-11-07 10:43:28 +02:00
Echo J	90e7a8d1d3	SilentPatch: Add missing cmath header include It's required for the modf() function (and it isn't implicitly included on MinGW GCC)	2024-11-07 10:43:28 +02:00
Echo J	915f227caf	SilentPatch: Move a header include in SVF This makes sure the fixed-width integer types are included in SVF.h	2024-11-07 10:43:28 +02:00
Echo J	9a0bc8a593	SilentPatch: Define a replacement for _stricmp() if needed MinGW GCC doesn't have this MSVC-specific function	2024-11-07 10:43:28 +02:00
Echo J	273e7c1790	SilentPatch: Move RwEngineInstance definition to separate headers MinGW GCC's linker will fail to find it in Common.cpp/SilentPatchVC.cpp otherwise	2024-11-07 10:43:27 +02:00
Echo J	976adef33f	SilentPatch: Simplify RwIm2DRenderLine function pointer declaration MinGW GCC doesn't seem to unwind the layers of the macro define properly (which causes it to not find the declaration type)	2024-11-07 10:28:52 +02:00
Echo J	ae61180c3b	SilentPatch: Don't define certain Rw* functions if not needed Redefining them can cause strange compile errors with MinGW GCC	2024-11-07 10:28:52 +02:00
Echo J	c57b5c543e	DDraw: Cast the memcpy() source argument This works around the MinGW GCC type strictness	2024-11-07 10:28:52 +02:00
Echo J	fabbe6e9d4	DDraw: Change the forwarded name for DirectDrawCreateEx MinGW GCC exports this function a bit differently (which doesn't work here)	2024-11-07 10:28:52 +02:00
Echo J	e49564a1c4	SilentPatchIII/VC: Mark some function pointers as inline MinGW GCC's linker can't find them otherwise	2024-11-07 10:28:52 +02:00
Echo J	4721fe05b3	SilentPatchIII/VC: Remove extern from ppUserFilesDir variable definitions This fixes compile warnings with MinGW GCC	2024-11-07 10:28:52 +02:00
Echo J	e8c1fde97f	SilentPatchIII/VC/SA: Add GCC-compatible safebuffers equivalent And switch to a common define for this attribute (this fixes compile warnings on MinGW GCC)	2024-11-07 10:28:52 +02:00
Echo J	464a5d729b	Use reinterpret_cast for function pointer casts MSVC (wrongly) allows those casts to succeed with static_cast: https://stackoverflow.com/questions/74002657/why-cant-i-static-cast-a-void-to-a-pointer-to-function (so adjust those casts for better compiler compatibility including MinGW GCC)	2024-11-07 10:28:52 +02:00
Echo J	886b660021	Set Windows defines earlier This avoids compile warnings on MinGW GCC (because standard C++ headers eventually import the Windows stuff)	2024-11-07 10:28:52 +02:00
Echo J	4966142826	Use lowercase names for Windows headers This fixes missing header issues on a case-sensitive filesystem with MinGW GCC	2024-11-07 10:28:52 +02:00