From 326a370bfa28d16f6d7a59d1941423ef1afeedc7 Mon Sep 17 00:00:00 2001
From: 3UR <76826837+3UR@users.noreply.github.com>
Date: Mon, 2 Mar 2026 10:11:19 +1000
Subject: [PATCH] chore: add a `MAX_TEXTURES` macro and change the naming
 convention of bool members

---
 Windows_Libs/Dev/Render/Renderer.h       |  8 +++++---
 Windows_Libs/Dev/Render/RendererCore.cpp | 16 ++++++++--------
 2 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/Windows_Libs/Dev/Render/Renderer.h b/Windows_Libs/Dev/Render/Renderer.h
index 92340ba..b218f69 100644
--- a/Windows_Libs/Dev/Render/Renderer.h
+++ b/Windows_Libs/Dev/Render/Renderer.h
@@ -5,6 +5,7 @@
 #include <vector>
 
 #define MAX_MIP_LEVELS 5
+#define MAX_TEXTURES 512
 
 class Renderer
 {
@@ -314,6 +315,7 @@ public:
 	struct Context
 	{
 		Context(ID3D11Device* device, ID3D11DeviceContext* deviceContext);
+
 		ID3D11DeviceContext* m_pDeviceContext;
 		ID3DUserDefinedAnnotation* userAnnotation;
 		DWORD contextStateFlags;
@@ -423,7 +425,7 @@ public:
 	C4JRender::eViewportType m_ViewportType;
 	BYTE reservedRendererByte0;
 	BYTE paddingAfterViewportType[3];
-	Renderer::Texture m_textures[512];
+	Renderer::Texture m_textures[MAX_TEXTURES];
 	DWORD backBufferWidth;
 	DWORD backBufferHeight;
 	BYTE reservedRendererByte1;
@@ -441,8 +443,8 @@ public:
 	std::unordered_map<int, ID3D11DepthStencilState*> managedDepthStencilStates;
 	std::unordered_map<int, ID3D11SamplerState*> managedSamplerStates;
 	std::unordered_map<int, ID3D11RasterizerState*> managedRasterizerStates;
-	BYTE shouldScreenGrabNextFrame;
-	BYTE suspended;
+	bool m_bShouldScreenGrabNextFrame;
+	BYTE m_bSuspended;
 	BYTE paddingAfterSuspendState[2];
 };
 
diff --git a/Windows_Libs/Dev/Render/RendererCore.cpp b/Windows_Libs/Dev/Render/RendererCore.cpp
index 9d2f666..08eaace 100644
--- a/Windows_Libs/Dev/Render/RendererCore.cpp
+++ b/Windows_Libs/Dev/Render/RendererCore.cpp
@@ -316,7 +316,7 @@ void Renderer::ConvertLinearToPng(ImageFileBuffer *pngOut,
   pngOut->m_bufferSize = outputLength;
 }
 
-void Renderer::DoScreenGrabOnNextPresent() { shouldScreenGrabNextFrame = 1; }
+void Renderer::DoScreenGrabOnNextPresent() { m_bShouldScreenGrabNextFrame = 1; }
 
 void Renderer::EndConditionalRendering() {}
 
@@ -371,8 +371,8 @@ void Renderer::Initialise(ID3D11Device *pDevice, IDXGISwapChain *pSwapChain) {
   }
   reservedRendererDword3 = 0;
 
-  shouldScreenGrabNextFrame = 0;
-  suspended = 0;
+  m_bShouldScreenGrabNextFrame = 0;
+  m_bSuspended = 0;
 
   this->SetupShaders();
   const float clearColour[4] = {0.0f, 0.0f, 0.0f, 0.0f};
@@ -522,7 +522,7 @@ bool Renderer::IsHiDef() { return true; }
 bool Renderer::IsWidescreen() { return true; }
 
 void Renderer::Present() {
-  if (shouldScreenGrabNextFrame) {
+  if (m_bShouldScreenGrabNextFrame) {
     int *linearData = new int[kScreenGrabWidth * kScreenGrabHeight];
     ID3D11Texture2D *backBuffer = nullptr;
     ID3D11Texture2D *stagingTexture = nullptr;
@@ -575,14 +575,14 @@ void Renderer::Present() {
       backBuffer->Release();
       backBuffer = nullptr;
     }
-    shouldScreenGrabNextFrame = 0;
+    m_bShouldScreenGrabNextFrame = 0;
   }
 
   m_pSwapChain->Present(1, 0);
   ++presentCount;
 }
 
-void Renderer::Resume() { suspended = 0; }
+void Renderer::Resume() { m_bSuspended = 0; }
 
 void Renderer::SetClearColour(const float colourRGBA[4]) {
   std::memcpy(m_fClearColor, colourRGBA, sizeof(m_fClearColor));
@@ -710,9 +710,9 @@ void Renderer::StartFrame() {
                                                depthStencilView);
 }
 
-void Renderer::Suspend() { suspended = 1; }
+void Renderer::Suspend() { m_bSuspended = 1; }
 
-bool Renderer::Suspended() { return suspended != 0; }
+bool Renderer::Suspended() { return m_bSuspended != 0; }
 
 void Renderer::UpdateGamma(unsigned short) {}