diff options
| author | 2018-08-20 19:22:43 -0400 | |
|---|---|---|
| committer | 2018-08-20 19:28:00 -0400 | |
| commit | bc16f7f3cce7b3a689f45697d9f6fbd970993e32 (patch) | |
| tree | 5bdf30d31fda49f68ec7e7ec6d9bb8f4d68db051 /src/video_core/renderer_base.cpp | |
| parent | Merge pull request #1104 from Subv/instanced_arrays (diff) | |
| download | yuzu-bc16f7f3cce7b3a689f45697d9f6fbd970993e32.tar.gz yuzu-bc16f7f3cce7b3a689f45697d9f6fbd970993e32.tar.xz yuzu-bc16f7f3cce7b3a689f45697d9f6fbd970993e32.zip | |
renderer_base: Make creation of the rasterizer, the responsibility of the renderers themselves
Given we use a base-class type within the renderer for the rasterizer
(RasterizerInterface), we want to allow renderers to perform more
complex initialization if they need to do such a thing. This makes it
important to reserve type information.
Given the OpenGL renderer is quite simple settings-wise, this is just a
simple shuffling of the initialization code. For something like Vulkan
however this might involve doing something like:
// Initialize and call rasterizer-specific function that requires
// the full type of the instance created.
auto raster = std::make_unique<VulkanRasterizer>(some, params);
raster->CallSomeVulkanRasterizerSpecificFunction();
// Assign to base class variable
rasterizer = std::move(raster)
Diffstat (limited to 'src/video_core/renderer_base.cpp')
| -rw-r--r-- | src/video_core/renderer_base.cpp | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/src/video_core/renderer_base.cpp b/src/video_core/renderer_base.cpp index afd86a83a..645d1521a 100644 --- a/src/video_core/renderer_base.cpp +++ b/src/video_core/renderer_base.cpp | |||
| @@ -2,7 +2,6 @@ | |||
| 2 | // Licensed under GPLv2 or any later version | 2 | // Licensed under GPLv2 or any later version |
| 3 | // Refer to the license.txt file included. | 3 | // Refer to the license.txt file included. |
| 4 | 4 | ||
| 5 | #include <memory> | ||
| 6 | #include "core/frontend/emu_window.h" | 5 | #include "core/frontend/emu_window.h" |
| 7 | #include "core/settings.h" | 6 | #include "core/settings.h" |
| 8 | #include "video_core/renderer_base.h" | 7 | #include "video_core/renderer_base.h" |
| @@ -17,18 +16,11 @@ RendererBase::RendererBase(Core::Frontend::EmuWindow& window) : render_window{wi | |||
| 17 | RendererBase::~RendererBase() = default; | 16 | RendererBase::~RendererBase() = default; |
| 18 | 17 | ||
| 19 | void RendererBase::RefreshBaseSettings() { | 18 | void RendererBase::RefreshBaseSettings() { |
| 20 | RefreshRasterizerSetting(); | ||
| 21 | UpdateCurrentFramebufferLayout(); | 19 | UpdateCurrentFramebufferLayout(); |
| 22 | 20 | ||
| 23 | renderer_settings.use_framelimiter = Settings::values.toggle_framelimit; | 21 | renderer_settings.use_framelimiter = Settings::values.toggle_framelimit; |
| 24 | } | 22 | } |
| 25 | 23 | ||
| 26 | void RendererBase::RefreshRasterizerSetting() { | ||
| 27 | if (rasterizer == nullptr) { | ||
| 28 | rasterizer = std::make_unique<RasterizerOpenGL>(render_window); | ||
| 29 | } | ||
| 30 | } | ||
| 31 | |||
| 32 | void RendererBase::UpdateCurrentFramebufferLayout() { | 24 | void RendererBase::UpdateCurrentFramebufferLayout() { |
| 33 | const Layout::FramebufferLayout& layout = render_window.GetFramebufferLayout(); | 25 | const Layout::FramebufferLayout& layout = render_window.GetFramebufferLayout(); |
| 34 | 26 | ||