VideoCommon: Rename Renderer s_ prefixes to m_

2 files changed, 89 insertions, 95 deletions

diff --git a/Source/Core/VideoCommon/RenderBase.cpp b/Source/Core/VideoCommon/RenderBase.cpp
index ab7a2adedf..fc93f45896 100644
--- a/Source/Core/VideoCommon/RenderBase.cpp
+++ b/Source/Core/VideoCommon/RenderBase.cpp
@@ -77,15 +77,13 @@ static float AspectToWidescreen(float aspect)
 }
 
 Renderer::Renderer(int backbuffer_width, int backbuffer_height)
+    : m_backbuffer_width(backbuffer_width), m_backbuffer_height(backbuffer_height),
+      m_last_efb_scale(g_ActiveConfig.iEFBScale)
 {
   FramebufferManagerBase::SetLastXfbWidth(MAX_XFB_WIDTH);
   FramebufferManagerBase::SetLastXfbHeight(MAX_XFB_HEIGHT);
 
   UpdateActiveConfig();
-
-  s_backbuffer_width = backbuffer_width;
-  s_backbuffer_height = backbuffer_height;
-  s_last_efb_scale = g_ActiveConfig.iEFBScale;
   CalculateTargetSize();
   UpdateDrawRectangle();
 
@@ -108,7 +106,7 @@ void Renderer::RenderToXFB(u32 xfbAddr, const EFBRectangle& sourceRc, u32 fbStri
   if (!fbStride || !fbHeight)
     return;
 
-  XFBWrited = true;
+  m_xfb_written = true;
 
   if (g_ActiveConfig.bUseXFB)
   {
@@ -132,7 +130,7 @@ int Renderer::EFBToScaledX(int x)
     return FramebufferManagerBase::ScaleToVirtualXfbWidth(x);
 
   default:
-    return x * (int)efb_scale_numeratorX / (int)efb_scale_denominatorX;
+    return x * (int)m_efb_scale_numeratorX / (int)m_efb_scale_denominatorX;
   };
 }
 
@@ -144,7 +142,7 @@ int Renderer::EFBToScaledY(int y)
     return FramebufferManagerBase::ScaleToVirtualXfbHeight(y);
 
   default:
-    return y * (int)efb_scale_numeratorY / (int)efb_scale_denominatorY;
+    return y * (int)m_efb_scale_numeratorY / (int)m_efb_scale_denominatorY;
   };
 }
 
@@ -157,8 +155,8 @@ void Renderer::CalculateTargetScale(int x, int y, int* scaledX, int* scaledY)
   }
   else
   {
-    *scaledX = x * (int)efb_scale_numeratorX / (int)efb_scale_denominatorX;
-    *scaledY = y * (int)efb_scale_numeratorY / (int)efb_scale_denominatorY;
+    *scaledX = x * (int)m_efb_scale_numeratorX / (int)m_efb_scale_denominatorX;
+    *scaledY = y * (int)m_efb_scale_numeratorY / (int)m_efb_scale_denominatorY;
   }
 }
 
@@ -169,70 +167,70 @@ bool Renderer::CalculateTargetSize()
   newEFBWidth = newEFBHeight = 0;
 
   // TODO: Ugly. Clean up
-  switch (s_last_efb_scale)
+  switch (m_last_efb_scale)
   {
   case SCALE_AUTO:
   case SCALE_AUTO_INTEGRAL:
     newEFBWidth = FramebufferManagerBase::ScaleToVirtualXfbWidth(EFB_WIDTH);
     newEFBHeight = FramebufferManagerBase::ScaleToVirtualXfbHeight(EFB_HEIGHT);
 
-    if (s_last_efb_scale == SCALE_AUTO_INTEGRAL)
+    if (m_last_efb_scale == SCALE_AUTO_INTEGRAL)
     {
-      efb_scale_numeratorX = efb_scale_numeratorY =
+      m_efb_scale_numeratorX = m_efb_scale_numeratorY =
           std::max((newEFBWidth - 1) / EFB_WIDTH + 1, (newEFBHeight - 1) / EFB_HEIGHT + 1);
-      efb_scale_denominatorX = efb_scale_denominatorY = 1;
+      m_efb_scale_denominatorX = m_efb_scale_denominatorY = 1;
       newEFBWidth = EFBToScaledX(EFB_WIDTH);
       newEFBHeight = EFBToScaledY(EFB_HEIGHT);
     }
     else
     {
-      efb_scale_numeratorX = newEFBWidth;
-      efb_scale_denominatorX = EFB_WIDTH;
-      efb_scale_numeratorY = newEFBHeight;
-      efb_scale_denominatorY = EFB_HEIGHT;
+      m_efb_scale_numeratorX = newEFBWidth;
+      m_efb_scale_denominatorX = EFB_WIDTH;
+      m_efb_scale_numeratorY = newEFBHeight;
+      m_efb_scale_denominatorY = EFB_HEIGHT;
     }
     break;
 
   case SCALE_1X:
-    efb_scale_numeratorX = efb_scale_numeratorY = 1;
-    efb_scale_denominatorX = efb_scale_denominatorY = 1;
+    m_efb_scale_numeratorX = m_efb_scale_numeratorY = 1;
+    m_efb_scale_denominatorX = m_efb_scale_denominatorY = 1;
     break;
 
   case SCALE_1_5X:
-    efb_scale_numeratorX = efb_scale_numeratorY = 3;
-    efb_scale_denominatorX = efb_scale_denominatorY = 2;
+    m_efb_scale_numeratorX = m_efb_scale_numeratorY = 3;
+    m_efb_scale_denominatorX = m_efb_scale_denominatorY = 2;
     break;
 
   case SCALE_2X:
-    efb_scale_numeratorX = efb_scale_numeratorY = 2;
-    efb_scale_denominatorX = efb_scale_denominatorY = 1;
+    m_efb_scale_numeratorX = m_efb_scale_numeratorY = 2;
+    m_efb_scale_denominatorX = m_efb_scale_denominatorY = 1;
     break;
 
   case SCALE_2_5X:
-    efb_scale_numeratorX = efb_scale_numeratorY = 5;
-    efb_scale_denominatorX = efb_scale_denominatorY = 2;
+    m_efb_scale_numeratorX = m_efb_scale_numeratorY = 5;
+    m_efb_scale_denominatorX = m_efb_scale_denominatorY = 2;
     break;
 
   default:
-    efb_scale_numeratorX = efb_scale_numeratorY = s_last_efb_scale - 3;
-    efb_scale_denominatorX = efb_scale_denominatorY = 1;
+    m_efb_scale_numeratorX = m_efb_scale_numeratorY = m_last_efb_scale - 3;
+    m_efb_scale_denominatorX = m_efb_scale_denominatorY = 1;
 
     const u32 max_size = GetMaxTextureSize();
-    if (max_size < EFB_WIDTH * efb_scale_numeratorX / efb_scale_denominatorX)
+    if (max_size < EFB_WIDTH * m_efb_scale_numeratorX / m_efb_scale_denominatorX)
     {
-      efb_scale_numeratorX = efb_scale_numeratorY = (max_size / EFB_WIDTH);
-      efb_scale_denominatorX = efb_scale_denominatorY = 1;
+      m_efb_scale_numeratorX = m_efb_scale_numeratorY = (max_size / EFB_WIDTH);
+      m_efb_scale_denominatorX = m_efb_scale_denominatorY = 1;
     }
 
     break;
   }
-  if (s_last_efb_scale > SCALE_AUTO_INTEGRAL)
+  if (m_last_efb_scale > SCALE_AUTO_INTEGRAL)
     CalculateTargetScale(EFB_WIDTH, EFB_HEIGHT, &newEFBWidth, &newEFBHeight);
 
-  if (newEFBWidth != s_target_width || newEFBHeight != s_target_height)
+  if (newEFBWidth != m_target_width || newEFBHeight != m_target_height)
   {
-    s_target_width = newEFBWidth;
-    s_target_height = newEFBHeight;
+    m_target_width = newEFBWidth;
+    m_target_height = newEFBHeight;
     return true;
   }
   return false;
@@ -261,25 +259,25 @@ void Renderer::ConvertStereoRectangle(const TargetRectangle& rc, TargetRectangle
   leftRc = drawRc, rightRc = drawRc;
   if (g_ActiveConfig.iStereoMode == STEREO_TAB)
   {
-    leftRc.top -= s_backbuffer_height / 4;
-    leftRc.bottom -= s_backbuffer_height / 4;
-    rightRc.top += s_backbuffer_height / 4;
-    rightRc.bottom += s_backbuffer_height / 4;
+    leftRc.top -= m_backbuffer_height / 4;
+    leftRc.bottom -= m_backbuffer_height / 4;
+    rightRc.top += m_backbuffer_height / 4;
+    rightRc.bottom += m_backbuffer_height / 4;
   }
   else
   {
-    leftRc.left -= s_backbuffer_width / 4;
-    leftRc.right -= s_backbuffer_width / 4;
-    rightRc.left += s_backbuffer_width / 4;
-    rightRc.right += s_backbuffer_width / 4;
+    leftRc.left -= m_backbuffer_width / 4;
+    leftRc.right -= m_backbuffer_width / 4;
+    rightRc.left += m_backbuffer_width / 4;
+    rightRc.right += m_backbuffer_width / 4;
   }
 }
 
 void Renderer::SetScreenshot(const std::string& filename)
 {
-  std::lock_guard<std::mutex> lk(s_criticalScreenshot);
-  s_sScreenshotName = filename;
-  s_screenshot.Set();
+  std::lock_guard<std::mutex> lk(m_screenshot_lock);
+  m_screenshot_name = filename;
+  m_screenshot_request.Set();
 }
 
 // Create On-Screen-Messages
@@ -432,7 +430,7 @@ float Renderer::CalculateDrawAspectRatio(int target_width, int target_height)
   {
     // If stretch is enabled, we prefer the aspect ratio of the window.
     return (static_cast<float>(target_width) / static_cast<float>(target_height)) /
-           (static_cast<float>(s_backbuffer_width) / static_cast<float>(s_backbuffer_height));
+           (static_cast<float>(m_backbuffer_width) / static_cast<float>(m_backbuffer_height));
   }
 
   // The rendering window aspect ratio as a proportion of the 4:3 or 16:9 ratio
@@ -477,8 +475,8 @@ TargetRectangle Renderer::CalculateFrameDumpDrawRectangle()
   if (!g_ActiveConfig.bInternalResolutionFrameDumps || g_ActiveConfig.RealXFBEnabled())
   {
     // But still remove the borders, since the caller expects this.
-    rc.right = target_rc.GetWidth();
-    rc.bottom = target_rc.GetHeight();
+    rc.right = m_target_rectangle.GetWidth();
+    rc.bottom = m_target_rectangle.GetHeight();
     return rc;
   }
 
@@ -496,8 +494,8 @@ TargetRectangle Renderer::CalculateFrameDumpDrawRectangle()
 
 void Renderer::UpdateDrawRectangle()
 {
-  float FloatGLWidth = static_cast<float>(s_backbuffer_width);
-  float FloatGLHeight = static_cast<float>(s_backbuffer_height);
+  float FloatGLWidth = static_cast<float>(m_backbuffer_width);
+  float FloatGLHeight = static_cast<float>(m_backbuffer_height);
   float FloatXOffset = 0;
   float FloatYOffset = 0;
 
@@ -556,7 +554,7 @@ void Renderer::UpdateDrawRectangle()
   // Check for force-settings and override.
 
   // The rendering window aspect ratio as a proportion of the 4:3 or 16:9 ratio
-  float Ratio = CalculateDrawAspectRatio(s_backbuffer_width, s_backbuffer_height);
+  float Ratio = CalculateDrawAspectRatio(m_backbuffer_width, m_backbuffer_height);
   if (g_ActiveConfig.iAspectRatio != ASPECT_STRETCH)
   {
     if (Ratio >= 0.995f && Ratio <= 1.005f)
@@ -610,10 +608,10 @@ void Renderer::UpdateDrawRectangle()
       iWhidth % 4;  // ensure divisibility by 4 to make it compatible with all the video encoders
   iHeight -= iHeight % 4;
 
-  target_rc.left = XOffset;
-  target_rc.top = YOffset;
-  target_rc.right = XOffset + iWhidth;
-  target_rc.bottom = YOffset + iHeight;
+  m_target_rectangle.left = XOffset;
+  m_target_rectangle.top = YOffset;
+  m_target_rectangle.right = XOffset + iWhidth;
+  m_target_rectangle.bottom = YOffset + iHeight;
 }
 
 void Renderer::InitializeCommon()
@@ -705,7 +703,7 @@ void Renderer::Swap(u32 xfbAddr, u32 fbWidth, u32 fbStride, u32 fbHeight, const
   // TODO: merge more generic parts into VideoCommon
   g_renderer->SwapImpl(xfbAddr, fbWidth, fbStride, fbHeight, rc, ticks, Gamma);
 
-  if (XFBWrited)
+  if (m_xfb_written)
     g_renderer->m_fps_counter.Update();
 
   frameCount++;
@@ -716,14 +714,14 @@ void Renderer::Swap(u32 xfbAddr, u32 fbWidth, u32 fbStride, u32 fbHeight, const
   // New frame
   stats.ResetFrame();
 
-  Core::Callback_VideoCopiedToXFB(XFBWrited ||
+  Core::Callback_VideoCopiedToXFB(m_xfb_written ||
                                   (g_ActiveConfig.bUseXFB && g_ActiveConfig.bUseRealXFB));
-  XFBWrited = false;
+  m_xfb_written = false;
 }
 
 bool Renderer::IsFrameDumping()
 {
-  if (s_screenshot.IsSet())
+  if (m_screenshot_request.IsSet())
     return true;
 
 #if defined(HAVE_LIBAV) || defined(_WIN32)
@@ -804,17 +802,17 @@ void Renderer::RunFrameDumps()
     }
 
     // Save screenshot
-    if (s_screenshot.TestAndClear())
+    if (m_screenshot_request.TestAndClear())
     {
-      std::lock_guard<std::mutex> lk(s_criticalScreenshot);
+      std::lock_guard<std::mutex> lk(m_screenshot_lock);
 
-      if (TextureToPng(config.data, config.stride, s_sScreenshotName, config.width, config.height,
+      if (TextureToPng(config.data, config.stride, m_screenshot_name, config.width, config.height,
                        false))
-        OSD::AddMessage("Screenshot saved to " + s_sScreenshotName);
+        OSD::AddMessage("Screenshot saved to " + m_screenshot_name);
 
       // Reset settings
-      s_sScreenshotName.clear();
-      s_screenshotCompleted.Set();
+      m_screenshot_name.clear();
+      s_screenshot_completed.Set();
     }
 
     if (SConfig::GetInstance().m_DumpFrames)
diff --git a/Source/Core/VideoCommon/RenderBase.h b/Source/Core/VideoCommon/RenderBase.h
index c8a7b05852..92ebade7db 100644
--- a/Source/Core/VideoCommon/RenderBase.h
+++ b/Source/Core/VideoCommon/RenderBase.h
@@ -85,11 +85,11 @@ public:
 
   // Ideal internal resolution - determined by display resolution (automatic scaling) and/or a
   // multiple of the native EFB resolution
-  int GetTargetWidth() { return s_target_width; }
-  int GetTargetHeight() { return s_target_height; }
+  int GetTargetWidth() { return m_target_width; }
+  int GetTargetHeight() { return m_target_height; }
   // Display resolution
-  int GetBackbufferWidth() { return s_backbuffer_width; }
-  int GetBackbufferHeight() { return s_backbuffer_height; }
+  int GetBackbufferWidth() { return m_backbuffer_width; }
+  int GetBackbufferHeight() { return m_backbuffer_height; }
   void SetWindowSize(int width, int height);
 
   // EFB coordinate conversion functions
@@ -97,7 +97,7 @@ public:
   // Use this to convert a whole native EFB rect to backbuffer coordinates
   virtual TargetRectangle ConvertEFBRectangle(const EFBRectangle& rc) = 0;
 
-  const TargetRectangle& GetTargetRectangle() { return target_rc; }
+  const TargetRectangle& GetTargetRectangle() { return m_target_rectangle; }
   float CalculateDrawAspectRatio(int target_width, int target_height);
   std::tuple<float, float> ScaleToDisplayAspectRatio(int width, int height);
   TargetRectangle CalculateFrameDumpDrawRectangle();
@@ -138,13 +138,13 @@ public:
   virtual void SwapImpl(u32 xfbAddr, u32 fbWidth, u32 fbStride, u32 fbHeight,
                         const EFBRectangle& rc, u64 ticks, float Gamma = 1.0f) = 0;
 
-  PEControl::PixelFormat GetPrevPixelFormat() { return prev_efb_format; }
-  void StorePixelFormat(PEControl::PixelFormat new_format) { prev_efb_format = new_format; }
+  PEControl::PixelFormat GetPrevPixelFormat() { return m_prev_efb_format; }
+  void StorePixelFormat(PEControl::PixelFormat new_format) { m_prev_efb_format = new_format; }
   PostProcessingShaderImplementation* GetPostProcessor() { return m_post_processor.get(); }
   // Max height/width
   virtual u32 GetMaxTextureSize() = 0;
 
-  Common::Event s_screenshotCompleted;
+  Common::Event s_screenshot_completed;
 
   // Final surface changing
   // This is called when the surface is resized (WX) or the window changes (Android).
@@ -161,24 +161,20 @@ protected:
                      bool swap_upside_down = false);
   void FinishFrameData();
 
-  Common::Flag s_screenshot;
-  std::mutex s_criticalScreenshot;
-  std::string s_sScreenshotName;
+  Common::Flag m_screenshot_request;
+  std::mutex m_screenshot_lock;
+  std::string m_screenshot_name;
 
   // The framebuffer size
-  int s_target_width = 0;
-  int s_target_height = 0;
+  int m_target_width = 0;
+  int m_target_height = 0;
 
   // TODO: Add functionality to reinit all the render targets when the window is resized.
-  int s_backbuffer_width = 0;
-  int s_backbuffer_height = 0;
-
-  TargetRectangle target_rc;
-
-  // TODO: Can probably eliminate this static var.
-  int s_last_efb_scale = 0;
-
-  bool XFBWrited = false;
+  int m_backbuffer_width = 0;
+  int m_backbuffer_height = 0;
+  int m_last_efb_scale = 0;
+  TargetRectangle m_target_rectangle;
+  bool m_xfb_written = false;
 
   FPSCounter m_fps_counter;
 
@@ -186,19 +182,19 @@ protected:
 
   static const float GX_MAX_DEPTH;
 
-  Common::Flag s_surface_needs_change;
-  Common::Event s_surface_changed;
-  void* s_new_surface_handle = nullptr;
+  Common::Flag m_surface_needs_change;
+  Common::Event m_surface_changed;
+  void* m_new_surface_handle = nullptr;
 
 private:
   void RunFrameDumps();
   void ShutdownFrameDumping();
 
-  PEControl::PixelFormat prev_efb_format = PEControl::INVALID_FMT;
-  unsigned int efb_scale_numeratorX = 1;
-  unsigned int efb_scale_numeratorY = 1;
-  unsigned int efb_scale_denominatorX = 1;
-  unsigned int efb_scale_denominatorY = 1;
+  PEControl::PixelFormat m_prev_efb_format = PEControl::INVALID_FMT;
+  unsigned int m_efb_scale_numeratorX = 1;
+  unsigned int m_efb_scale_numeratorY = 1;
+  unsigned int m_efb_scale_denominatorX = 1;
+  unsigned int m_efb_scale_denominatorY = 1;
 
   // frame dumping
   std::thread m_frame_dump_thread;