From 7d9eac76600e4e7c9989e6773cd33e478dbb85ba Mon Sep 17 00:00:00 2001 From: Lynix Date: Mon, 26 Aug 2013 13:36:41 +0200 Subject: [PATCH] Refactored code Former-commit-id: 95c723425cc2ac626b5ddd6d77f69e0a4b0a630a --- .../Nazara/Graphics/ForwardRenderTechnique.hpp | 6 +++--- src/Nazara/Graphics/ForwardRenderTechnique.cpp | 18 +++++++++--------- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/include/Nazara/Graphics/ForwardRenderTechnique.hpp b/include/Nazara/Graphics/ForwardRenderTechnique.hpp index cb0638646..9aa3a9ce1 100644 --- a/include/Nazara/Graphics/ForwardRenderTechnique.hpp +++ b/include/Nazara/Graphics/ForwardRenderTechnique.hpp @@ -29,9 +29,9 @@ class NAZARA_API NzForwardRenderTechnique : public NzAbstractRenderTechnique void SetMaxLightsPerObject(unsigned int lightCount); private: - void DrawOpaqueModels(const NzScene* scene, NzForwardRenderQueue::BatchedModelContainer& opaqueModels); - void DrawSprites(const NzScene* scene, NzForwardRenderQueue::BatchedSpriteContainer& sprites); - void DrawTransparentModels(const NzScene* scene, NzForwardRenderQueue::TransparentModelContainer& transparentModels); + void DrawOpaqueModels(const NzScene* scene); + void DrawSprites(const NzScene* scene); + void DrawTransparentModels(const NzScene* scene); NzForwardRenderQueue m_renderQueue; NzIndexBufferRef m_indexBuffer; diff --git a/src/Nazara/Graphics/ForwardRenderTechnique.cpp b/src/Nazara/Graphics/ForwardRenderTechnique.cpp index 91dedf4fb..8f0dc0342 100644 --- a/src/Nazara/Graphics/ForwardRenderTechnique.cpp +++ b/src/Nazara/Graphics/ForwardRenderTechnique.cpp @@ -81,13 +81,13 @@ void NzForwardRenderTechnique::Draw(const NzScene* scene) m_renderQueue.Sort(scene->GetViewer()); if (!m_renderQueue.opaqueModels.empty()) - DrawOpaqueModels(scene, m_renderQueue.opaqueModels); + DrawOpaqueModels(scene); if (!m_renderQueue.sprites.empty()) - DrawSprites(scene, m_renderQueue.sprites); + DrawSprites(scene); if (!m_renderQueue.transparentsModels.empty()) - DrawTransparentModels(scene, m_renderQueue.transparentsModels); + DrawTransparentModels(scene); // Les autres drawables (Exemple: Terrain) for (const NzDrawable* drawable : m_renderQueue.otherDrawables) @@ -162,14 +162,14 @@ void NzForwardRenderTechnique::SetMaxLightsPerObject(unsigned int lightCount) m_maxLightsPerObject = lightCount; } -void NzForwardRenderTechnique::DrawOpaqueModels(const NzScene* scene, NzForwardRenderQueue::BatchedModelContainer& opaqueModels) +void NzForwardRenderTechnique::DrawOpaqueModels(const NzScene* scene) { NzAbstractViewer* viewer = scene->GetViewer(); const NzShaderProgram* lastProgram = nullptr; unsigned int lightCount = 0; - for (auto& matIt : opaqueModels) + for (auto& matIt : m_renderQueue.opaqueModels) { bool& used = std::get<0>(matIt.second); if (used) @@ -320,7 +320,7 @@ void NzForwardRenderTechnique::DrawOpaqueModels(const NzScene* scene, NzForwardR } } -void NzForwardRenderTechnique::DrawSprites(const NzScene* scene, NzForwardRenderQueue::BatchedSpriteContainer& sprites) +void NzForwardRenderTechnique::DrawSprites(const NzScene* scene) { NzAbstractViewer* viewer = scene->GetViewer(); const NzShaderProgram* lastProgram = nullptr; @@ -329,7 +329,7 @@ void NzForwardRenderTechnique::DrawSprites(const NzScene* scene, NzForwardRender NzRenderer::SetMatrix(nzMatrixType_World, NzMatrix4f::Identity()); NzRenderer::SetVertexBuffer(&m_spriteBuffer); - for (auto& matIt : sprites) + for (auto& matIt : m_renderQueue.sprites) { const NzMaterial* material = matIt.first; auto& spriteVector = matIt.second; @@ -400,13 +400,13 @@ void NzForwardRenderTechnique::DrawSprites(const NzScene* scene, NzForwardRender } } -void NzForwardRenderTechnique::DrawTransparentModels(const NzScene* scene, NzForwardRenderQueue::TransparentModelContainer& transparentModels) +void NzForwardRenderTechnique::DrawTransparentModels(const NzScene* scene) { NzAbstractViewer* viewer = scene->GetViewer(); const NzShaderProgram* lastProgram = nullptr; unsigned int lightCount = 0; - for (const std::pair& pair : transparentModels) + for (const std::pair& pair : m_renderQueue.transparentsModels) { // Matériau const NzMaterial* material = (pair.second) ?