sandbox:(VERA): prefer smart pointers over raw pointers

sandbox.cpp: * Made cursor vbo use std::unique_ptr.
patriciogonzalezvivo · Jul 29, 2022 · 6db639f · 6db639f
1 parent 065eb13
commit 6db639f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/src/sandbox.cpp b/src/sandbox.cpp
@@ -1732,8 +1732,8 @@ void Sandbox::renderUI() {
 
     if (cursor && vera::getMouseEntered()) {
         // TRACK_BEGIN("cursor")
-        if (m_cross_vbo == nullptr) 
-            m_cross_vbo = new vera::Vbo( vera::crossMesh( glm::vec3(0.0f, 0.0f, 0.0f), 10.0f) );
+        if (m_cross_vbo == nullptr)
+            m_cross_vbo = std::unique_ptr<vera::Vbo>(new vera::Vbo( vera::crossMesh( glm::vec3(0.0f, 0.0f, 0.0f), 10.0f) ));
 
         vera::Shader* fill = vera::getFillShader();
         fill->use();
@@ -1784,9 +1784,6 @@ void Sandbox::clear() {
 
     if (uniforms.models.size() > 0)
         m_sceneRender.clear();
-
-    if (m_cross_vbo)
-        delete m_cross_vbo;
 }
 
 void Sandbox::printDependencies(ShaderType _type) const {

diff --git a/src/sandbox.h b/src/sandbox.h
@@ -127,7 +127,7 @@ class Sandbox {
     bool                m_postprocessing;
 
     // Cursor
-    vera::Vbo*          m_cross_vbo;
+    std::unique_ptr<vera::Vbo>          m_cross_vbo;
 
     // debug plot texture and shader for histogram or fps plots
     vera::Shader        m_plot_shader;