Use std::unique_ptr for all MessageContext

16 files changed, 18 insertions, 22 deletions

diff --git a/src/ui/tools/arc-tool.cpp b/src/ui/tools/arc-tool.cpp
index 18bd36361..b10b8904c 100644
--- a/src/ui/tools/arc-tool.cpp
+++ b/src/ui/tools/arc-tool.cpp
@@ -320,7 +320,7 @@ bool ArcTool::root_handler(GdkEvent* event) {
 
 void ArcTool::drag(Geom::Point pt, guint state) {
     if (!this->arc) {
-        if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+        if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
             return;
         }
 
diff --git a/src/ui/tools/box3d-tool.cpp b/src/ui/tools/box3d-tool.cpp
index 76dba4eea..203e224a3 100644
--- a/src/ui/tools/box3d-tool.cpp
+++ b/src/ui/tools/box3d-tool.cpp
@@ -496,7 +496,7 @@ bool Box3dTool::root_handler(GdkEvent* event) {
 
 void Box3dTool::drag(guint /*state*/) {
     if (!this->box3d) {
-        if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+        if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
             return;
         }
 
diff --git a/src/ui/tools/calligraphic-tool.cpp b/src/ui/tools/calligraphic-tool.cpp
index 80031cffc..5ce2c4813 100644
--- a/src/ui/tools/calligraphic-tool.cpp
+++ b/src/ui/tools/calligraphic-tool.cpp
@@ -447,7 +447,7 @@ bool CalligraphicTool::root_handler(GdkEvent* event) {
     switch (event->type) {
         case GDK_BUTTON_PRESS:
             if (event->button.button == 1 && !this->space_panning) {
-                if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+                if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
                     return TRUE;
                 }
 
diff --git a/src/ui/tools/connector-tool.cpp b/src/ui/tools/connector-tool.cpp
index 4b2340913..95b13f92c 100644
--- a/src/ui/tools/connector-tool.cpp
+++ b/src/ui/tools/connector-tool.cpp
@@ -470,7 +470,7 @@ bool ConnectorTool::_handleButtonPress(GdkEventButton const &bevent)
     bool ret = false;
 
         if ( bevent.button == 1 && !this->space_panning ) {
-            if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+            if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
                 return true;
             }
 
diff --git a/src/ui/tools/dynamic-base.h b/src/ui/tools/dynamic-base.h
index d9c1f9804..1a36f302c 100644
--- a/src/ui/tools/dynamic-base.h
+++ b/src/ui/tools/dynamic-base.h
@@ -102,8 +102,6 @@ protected:
     double tremor;
     double cap_rounding;
 
-    //Inkscape::MessageContext *_message_context;
-
     bool is_drawing;
 
     /** uses absolute width independent of zoom */
diff --git a/src/ui/tools/eraser-tool.cpp b/src/ui/tools/eraser-tool.cpp
index 4f9e3de08..36e6a44aa 100644
--- a/src/ui/tools/eraser-tool.cpp
+++ b/src/ui/tools/eraser-tool.cpp
@@ -384,7 +384,7 @@ bool EraserTool::root_handler(GdkEvent* event) {
     switch (event->type) {
         case GDK_BUTTON_PRESS:
             if (event->button.button == 1 && !this->space_panning) {
-                if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+                if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
                     return TRUE;
                 }
 
diff --git a/src/ui/tools/pen-tool.cpp b/src/ui/tools/pen-tool.cpp
index d9feb89c2..89d100c71 100644
--- a/src/ui/tools/pen-tool.cpp
+++ b/src/ui/tools/pen-tool.cpp
@@ -390,7 +390,7 @@ bool PenTool::_handleButtonPress(GdkEventButton const &bevent) {
         // make sure this is not the last click for a waiting LPE (otherwise we want to finish the path)
         && this->expecting_clicks_for_LPE != 1) {
 
-        if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+        if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
             return true;
         }
 
diff --git a/src/ui/tools/pencil-tool.cpp b/src/ui/tools/pencil-tool.cpp
index eac7c585f..8e1966f4c 100644
--- a/src/ui/tools/pencil-tool.cpp
+++ b/src/ui/tools/pencil-tool.cpp
@@ -178,7 +178,7 @@ bool PencilTool::_handleButtonPress(GdkEventButton const &bevent) {
     if ( bevent.button == 1  && !this->space_panning) {
         Inkscape::Selection *selection = desktop->getSelection();
 
-        if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+        if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
             return true;
         }
 
diff --git a/src/ui/tools/rect-tool.cpp b/src/ui/tools/rect-tool.cpp
index f6387d6dc..4000e4924 100644
--- a/src/ui/tools/rect-tool.cpp
+++ b/src/ui/tools/rect-tool.cpp
@@ -362,7 +362,7 @@ void RectTool::drag(Geom::Point const pt, guint state) {
     SPDesktop *desktop = this->desktop;
 
     if (!this->rect) {
-        if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+        if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
             return;
         }
 
diff --git a/src/ui/tools/select-tool.h b/src/ui/tools/select-tool.h
index 12e4b1aa7..c721771e2 100644
--- a/src/ui/tools/select-tool.h
+++ b/src/ui/tools/select-tool.h
@@ -21,7 +21,6 @@
 struct SPCanvasItem;
 
 namespace Inkscape {
-  class MessageContext;
   class SelTrans;
   class SelectionDescriber;
 }
diff --git a/src/ui/tools/spiral-tool.cpp b/src/ui/tools/spiral-tool.cpp
index f05f54fca..c1620d919 100644
--- a/src/ui/tools/spiral-tool.cpp
+++ b/src/ui/tools/spiral-tool.cpp
@@ -331,7 +331,7 @@ void SpiralTool::drag(Geom::Point const &p, guint state) {
     int const snaps = prefs->getInt("/options/rotationsnapsperpi/value", 12);
 
     if (!this->spiral) {
-        if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+        if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
             return;
         }
 
diff --git a/src/ui/tools/spray-tool.cpp b/src/ui/tools/spray-tool.cpp
index fd13b4d3d..91dedd742 100644
--- a/src/ui/tools/spray-tool.cpp
+++ b/src/ui/tools/spray-tool.cpp
@@ -1232,7 +1232,7 @@ bool SprayTool::root_handler(GdkEvent* event) {
             break;
         case GDK_BUTTON_PRESS:
             if (event->button.button == 1 && !this->space_panning) {
-                if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+                if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
                     return TRUE;
                 }
                 this->setCloneTilerPrefs();
@@ -1307,7 +1307,7 @@ bool SprayTool::root_handler(GdkEvent* event) {
                     case GDK_SCROLL_DOWN:
                     case GDK_SCROLL_UP:
                     case GDK_SCROLL_SMOOTH: {
-                        if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+                        if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
                             return TRUE;
                         }
                         this->last_push = desktop->dt2doc(scroll_dt);
diff --git a/src/ui/tools/star-tool.cpp b/src/ui/tools/star-tool.cpp
index ce0fa9aef..2e92d1ae9 100644
--- a/src/ui/tools/star-tool.cpp
+++ b/src/ui/tools/star-tool.cpp
@@ -342,7 +342,7 @@ void StarTool::drag(Geom::Point p, guint state)
     int const snaps = prefs->getInt("/options/rotationsnapsperpi/value", 12);
 
     if (!this->star) {
-        if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+        if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
             return;
         }
 
diff --git a/src/ui/tools/tool-base.cpp b/src/ui/tools/tool-base.cpp
index a10165554..624c4df4a 100644
--- a/src/ui/tools/tool-base.cpp
+++ b/src/ui/tools/tool-base.cpp
@@ -119,9 +119,7 @@ ToolBase::ToolBase(gchar const *const *cursor_shape, bool uses_snap)
 }
 
 ToolBase::~ToolBase() {
-    if (this->message_context) {
-        delete this->message_context;
-    }
+    this->message_context = nullptr;
 
     if (this->desktop) {
         this->desktop = nullptr;
diff --git a/src/ui/tools/tool-base.h b/src/ui/tools/tool-base.h
index e43b347f0..3533482cf 100644
--- a/src/ui/tools/tool-base.h
+++ b/src/ui/tools/tool-base.h
@@ -15,6 +15,7 @@
 
 #include <cstddef>
 #include <string>
+#include <memory>
 #include "knot.h"
 #include "knotholder.h"
 #include <2geom/point.h>
@@ -162,11 +163,11 @@ public:
     SPItem *item_to_select; ///< the item where mouse_press occurred, to
                             ///< be selected if this is a click not drag
 
-    Inkscape::MessageContext *defaultMessageContext() {
-        return message_context;
+    Inkscape::MessageContext *defaultMessageContext() const {
+        return message_context.get();
     }
 
-    Inkscape::MessageContext *message_context;
+    std::unique_ptr<Inkscape::MessageContext> message_context;
 
     Inkscape::SelCue *_selcue;
 
diff --git a/src/ui/tools/tweak-tool.cpp b/src/ui/tools/tweak-tool.cpp
index 3f530c278..cbc3dfd99 100644
--- a/src/ui/tools/tweak-tool.cpp
+++ b/src/ui/tools/tweak-tool.cpp
@@ -1156,7 +1156,7 @@ bool TweakTool::root_handler(GdkEvent* event) {
         case GDK_BUTTON_PRESS:
             if (event->button.button == 1 && !this->space_panning) {
 
-                if (Inkscape::have_viable_layer(desktop, this->message_context) == false) {
+                if (Inkscape::have_viable_layer(desktop, defaultMessageContext()) == false) {
                     return TRUE;
                 }