Factor out duplicate code between initVideo and setResolution

nxengine · Feb 6, 2023 · 486f979 · 486f979
1 parent fb1e77d
commit 486f979
Showing 1 changed file with 19 additions and 34 deletions.
diff --git a/src/graphics/Renderer.cpp b/src/graphics/Renderer.cpp
@@ -69,15 +69,6 @@ bool Renderer::initVideo(int scale)
 {
   uint32_t window_flags = SDL_WINDOW_SHOWN | SDL_WINDOW_ALLOW_HIGHDPI;
 
-  //TODO: sync this with setResolution
-  if (widescreen) {
-    screenWidth  = 432;
-    screenHeight = 243;
-  } else {
-    screenWidth  = 320;
-    screenHeight = 240;
-  }
-
   if (_window)
   {
     LOG_WARN("second call to Renderer::InitVideo()");
@@ -133,13 +124,7 @@ bool Renderer::initVideo(int scale)
 
   LOG_INFO("Renderer::initVideo: using: {} renderer", info.name);
 
-  if (SDL_RenderSetLogicalSize(_renderer, screenWidth, screenHeight))
-  {
-    LOG_ERROR("Renderer::initVideo: SDL_RenderSetLogicalSize failed: {}", SDL_GetError());
-    return false;
-  }
-
-  if (!createRenderTarget(screenWidth, screenHeight))
+  if (!setResolution(scale, widescreen))
     return false;
 
   std::string spotpath = ResourceManager::getInstance()->getPath("spot.png");
@@ -152,24 +137,6 @@ bool Renderer::initVideo(int scale)
   return true;
 }
 
-bool Renderer::createRenderTarget(int width, int height)
-{
-  SDL_RendererInfo info;
-  SDL_GetRendererInfo(_renderer, &info);
-
-  _texture = SDL_CreateTexture(_renderer,
-      info.texture_formats[0],
-      SDL_TEXTUREACCESS_TARGET,
-      width, height);
-  if (SDL_SetRenderTarget(_renderer, _texture)) {
-    LOG_ERROR("Renderer::createRenderTarget: SDL_SetRenderTarget failed: {}", SDL_GetError());
-    return false;
-  }
-  SDL_RenderClear(_renderer);
-
-  return true;
-}
-
 bool Renderer::flushAll()
 {
   LOG_DEBUG("Renderer::flushAll()");
@@ -255,6 +222,24 @@ int Renderer::getResolutionCount()
 #endif
 }
 
+bool Renderer::createRenderTarget(int width, int height)
+{
+  SDL_RendererInfo info;
+  SDL_GetRendererInfo(_renderer, &info);
+
+  _texture = SDL_CreateTexture(_renderer,
+      info.texture_formats[0],
+      SDL_TEXTUREACCESS_TARGET,
+      width, height);
+  if (SDL_SetRenderTarget(_renderer, _texture)) {
+    LOG_ERROR("Renderer::createRenderTarget: SDL_SetRenderTarget failed: {}", SDL_GetError());
+    return false;
+  }
+  SDL_RenderClear(_renderer);
+
+  return true;
+}
+
 void Renderer::showLoadingScreen()
 {
   Surface loading;