Applied defer to texture file load, implemented relevant unit tests

Author: jonjondev

engine/render/renderer/platform/vulkan/Texture2D.cpp

@@ -15,6 +15,7 @@
 
 #define STB_IMAGE_IMPLEMENTATION
 #include <stb_image.h>
+#include <utils/Defer.h>
 #include <utils/Logging.h>
 
 namespace Siege::Vulkan
@@ -73,6 +74,7 @@ void Texture2D::LoadFromFile(const char* filePath)
     uint64_t imageSize = texWidth * texHeight * 4;
 
     uint8_t* pixelPtr = new uint8_t[imageSize];
+    defer([pixelPtr] { delete[] pixelPtr; });
 
     // Set the image to 0 so that we get no garbage data in the pixel array
     memset(pixelPtr, 0, imageSize);
@@ -102,8 +104,6 @@ void Texture2D::LoadFromFile(const char* filePath)
     image.CopyBuffer(stagingBuffer.buffer);
 
     Buffer::DestroyBuffer(stagingBuffer);
-
-    delete[] pixelPtr;
 }
 
 void Texture2D::LoadTexture(const uint8_t* pixels, size_t size, uint32_t width, uint32_t height)

tests/utils/test_Defer.cpp

@@ -0,0 +1,49 @@
+//
+// Copyright (c) 2022 Jonathan Moallem (@J-Mo63) & Aryeh Zinn (@Raelr)
+//
+// This code is released under an unmodified zlib license.
+// For conditions of distribution and use, please see:
+//     https://opensource.org/licenses/Zlib
+//
+
+#include <utest.h>
+#include <utils/Defer.h>
+
+UTEST(test_Defer, BasicScopeDeferModify)
+{
+    bool a = true;
+    {
+        ASSERT_TRUE(a);
+        defer([&a] { a = false; });
+        a = true;
+        ASSERT_TRUE(a);
+    }
+    ASSERT_FALSE(a);
+}
+
+UTEST(test_Defer, BasicScopeDeferDeletion)
+{
+    auto* a = new bool;
+    {
+        ASSERT_TRUE(a);
+        defer([&a] {
+            delete a;
+            a = nullptr;
+        });
+        ASSERT_TRUE(a);
+    }
+    ASSERT_FALSE(a);
+}
+
+UTEST(test_Defer, LoopScopeDeferModify)
+{
+    bool a = true;
+    for (uint8_t i = 0; i < 3; ++i)
+    {
+        ASSERT_TRUE(a);
+        defer([&a] { a = true; });
+        a = false;
+        ASSERT_FALSE(a);
+    }
+    ASSERT_TRUE(a);
+}