dolphin/Source/Core/VideoCommon/FramebufferManagerBase.cpp

// Copyright 2010 Dolphin Emulator Project
// Licensed under GPLv2+
// Refer to the license.txt file included.

#include "VideoCommon/FramebufferManagerBase.h"

#include <memory>

#include "VideoCommon/RenderBase.h"

std::unique_ptr<FramebufferManagerBase> g_framebuffer_manager;

unsigned int FramebufferManagerBase::m_EFBLayers = 1;

FramebufferManagerBase::~FramebufferManagerBase() = default;
Add missing license headers 2015-05-24 06:32:32 +02:00			`// Copyright 2010 Dolphin Emulator Project`
			`// Licensed under GPLv2+`
			`// Refer to the license.txt file included.`

Reformat all the things. Have fun with merge conflicts. 2016-06-24 10:43:46 +02:00			`#include "VideoCommon/FramebufferManagerBase.h"`
FramebufferManagerBase: Return a std::pair from GetTargetSize Keeps associated data together. It also eliminates the possibility of out parameters not being initialized properly. For example, consider the following example: -- some FramebufferManager implementation -- void FBMgrImpl::GetTargetSize(u32* width, u32* height) override { // Do nothing } -- somewhere else where the function is used -- u32 width, height; framebuffer_manager_instance->GetTargetSize(&width, &height); if (texture_width != width) <-- Uninitialized variable usage { ... } It makes it much more obvious to spot any initialization issues, because it requires something to be returned, as opposed to allowing an implementation to just not do anything. 2017-02-03 12:31:20 -05:00
FramebufferManagerBase: Get rid of explicit delete and new 2015-12-21 14:11:01 -05:00			`#include <memory>`
FramebufferManagerBase: Return a std::pair from GetTargetSize Keeps associated data together. It also eliminates the possibility of out parameters not being initialized properly. For example, consider the following example: -- some FramebufferManager implementation -- void FBMgrImpl::GetTargetSize(u32* width, u32* height) override { // Do nothing } -- somewhere else where the function is used -- u32 width, height; framebuffer_manager_instance->GetTargetSize(&width, &height); if (texture_width != width) <-- Uninitialized variable usage { ... } It makes it much more obvious to spot any initialization issues, because it requires something to be returned, as opposed to allowing an implementation to just not do anything. 2017-02-03 12:31:20 -05:00
Convert all includes to relative paths. 2014-02-17 05:18:15 -05:00			`#include "VideoCommon/RenderBase.h"`
Some compiler warnings and eol-style. git-svn-id: https://dolphin-emu.googlecode.com/svn/trunk@6418 8ced0084-cf51-0410-be5f-012b33b47a6e 2010-11-14 23:56:26 +00:00
Render: Get rid of explicit new and delete 2015-12-22 18:47:20 -05:00			`std::unique_ptr<FramebufferManagerBase> g_framebuffer_manager;`
Some compiler warnings and eol-style. git-svn-id: https://dolphin-emu.googlecode.com/svn/trunk@6418 8ced0084-cf51-0410-be5f-012b33b47a6e 2010-11-14 23:56:26 +00:00
TextureCache: Ensure that all render target textures have as many layers as the frame buffer. Also fixes a case where the D3D code path did not initialize num_layers leading to undefined behaviour. 2014-11-06 11:41:39 +01:00			`unsigned int FramebufferManagerBase::m_EFBLayers = 1;`

Remove old XFB logic 2017-08-06 23:05:42 -05:00			`FramebufferManagerBase::~FramebufferManagerBase() = default;`