From 0f0f0cbb268347c005042834192ce8bf46c6c50a Mon Sep 17 00:00:00 2001 From: Sergey Sharybin Date: Mon, 20 Feb 2023 15:27:01 +0100 Subject: [PATCH] Cleanup: Remove check for this pointer not being nullptr The check was triggering the 'this' pointer cannot be null in well-defined C++ code We do not check for this pointer in any other areas. If it is needed due to possible opaque pointer cast to the check prior to the cast. --- source/blender/gpu/metal/mtl_framebuffer.mm | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/source/blender/gpu/metal/mtl_framebuffer.mm b/source/blender/gpu/metal/mtl_framebuffer.mm index 9b334dd7166..5e3a4136d88 100644 --- a/source/blender/gpu/metal/mtl_framebuffer.mm +++ b/source/blender/gpu/metal/mtl_framebuffer.mm @@ -1754,9 +1754,8 @@ void MTLFrameBuffer::blit(uint read_slot, uint height, eGPUFrameBufferBits blit_buffers) { - BLI_assert(this); BLI_assert(metal_fb_write); - if (!(this && metal_fb_write)) { + if (!metal_fb_write) { return; } MTLContext *mtl_context = reinterpret_cast(GPU_context_active_get()); @@ -1899,4 +1898,4 @@ int MTLFrameBuffer::get_height() return height_; } -} // blender::gpu +} // namespace blender::gpu -- 2.30.2