[cleanup] Use std::unique_ptr for Tab

kallisti5 · Oct 15, 2024 · c758bd4 · c758bd4
1 parent ef7b702
commit c758bd4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 10 deletions.
diff --git a/include/guisan/widgets/tab.hpp b/include/guisan/widgets/tab.hpp
@@ -57,13 +57,14 @@
 #ifndef GCN_TAB_HPP
 #define GCN_TAB_HPP
 
-#include <map>
-#include <string>
-
 #include "guisan/mouselistener.hpp"
 #include "guisan/platform.hpp"
 #include "guisan/widget.hpp"
 
+#include <map>
+#include <memory>
+#include <string>
+
 namespace gcn
 {
     class Label;
@@ -144,7 +145,7 @@ namespace gcn
         /**
          * Holds the label of the tab.
          */
-        Label* mLabel = nullptr;
+        std::unique_ptr<Label> mLabel;
 
         /**
          * Holds the tabbed area the tab is a part of.

diff --git a/src/widgets/tab.cpp b/src/widgets/tab.cpp
@@ -68,19 +68,16 @@
 
 namespace gcn
 {
-    Tab::Tab() : mLabel(new Label())
+    Tab::Tab() : mLabel(std::make_unique<Label>())
     {
         mLabel->setPosition(6, 6);
-        add(mLabel);
+        add(mLabel.get());
         setFrameSize(1);
 
         addMouseListener(this);
     }
 
-    Tab::~Tab()
-    {
-        delete mLabel;
-    }
+    Tab::~Tab() = default;
 
     void Tab::adjustSize()
     {