From a45c2229411fa473dd3a619b214707febe4fc795 Mon Sep 17 00:00:00 2001 From: baldurk Date: Tue, 8 May 2018 12:10:19 +0100 Subject: [PATCH] [Coverity] Remove redundant if() comparisons --- qrenderdoc/Windows/PipelineState/GLPipelineStateViewer.cpp | 3 --- renderdoc/driver/d3d11/d3d11_context_wrap.cpp | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/qrenderdoc/Windows/PipelineState/GLPipelineStateViewer.cpp b/qrenderdoc/Windows/PipelineState/GLPipelineStateViewer.cpp index 92da6daf5..81fa9f26b 100644 --- a/qrenderdoc/Windows/PipelineState/GLPipelineStateViewer.cpp +++ b/qrenderdoc/Windows/PipelineState/GLPipelineStateViewer.cpp @@ -839,9 +839,6 @@ void GLPipelineStateViewer::setShaderState(const GLPipe::Shader &stage, RDLabel node->setTag(QVariant::fromValue(idx)); - if(!filledSlot) - setEmptyRow(node); - if(!usedSlot) setInactiveRow(node); diff --git a/renderdoc/driver/d3d11/d3d11_context_wrap.cpp b/renderdoc/driver/d3d11/d3d11_context_wrap.cpp index fc36fceee..7e2ea4817 100644 --- a/renderdoc/driver/d3d11/d3d11_context_wrap.cpp +++ b/renderdoc/driver/d3d11/d3d11_context_wrap.cpp @@ -3458,7 +3458,7 @@ void WrappedID3D11DeviceContext::OMSetRenderTargetsAndUnorderedAccessViews( // log (which might make this combination valid on replay without some of the targets!) for(UINT i = 0; i < NumUAVs; i++) { - if(UAVs && UAVs[i]) + if(UAVs[i]) { MarkResourceReferenced(GetIDForResource(UAVs[i]), eFrameRef_Read); MarkResourceReferenced(GetViewResourceResID(UAVs[i]), eFrameRef_Read);