From e32d1aa28a50a18c632af1bf3cb3e6c78fcd3144 Mon Sep 17 00:00:00 2001
From: wmayer <wmayer@users.sourceforge.net>
Date: Tue, 14 May 2024 10:55:41 +0200
Subject: [PATCH] TD: Fix several compiler warnings

---
 src/Mod/TechDraw/Gui/MDIViewPage.cpp     | 11 ++++-------
 src/Mod/TechDraw/Gui/TechDrawHandler.cpp |  2 +-
 2 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/src/Mod/TechDraw/Gui/MDIViewPage.cpp b/src/Mod/TechDraw/Gui/MDIViewPage.cpp
index 7781da3a20..aefe3d5e9c 100644
--- a/src/Mod/TechDraw/Gui/MDIViewPage.cpp
+++ b/src/Mod/TechDraw/Gui/MDIViewPage.cpp
@@ -807,10 +807,10 @@ void MDIViewPage::setTreeToSceneSelect()
             auto* vert = dynamic_cast<QGIVertex*>(scene);
             auto* face = dynamic_cast<QGIFace*>(scene);
             if (edge || vert || face) {
-                const char* ssn = getSceneSubName(scene).c_str();
+                std::string ssn = getSceneSubName(scene);
 
-                Gui::Selection().addSelection(doc_name, obj_name, ssn);
-                showStatusMsg(doc_name, obj_name, ssn);
+                Gui::Selection().addSelection(doc_name, obj_name, ssn.c_str());
+                showStatusMsg(doc_name, obj_name, ssn.c_str());
                 return;
             }
             else if (dynamic_cast<QGIDatumLabel*>(scene) || dynamic_cast<QGMText*>(scene)) {
@@ -846,8 +846,6 @@ std::string MDIViewPage::getSceneSubName(QGraphicsItem* scene)
     if (edge || vert || face) {
         auto* viewItem = dynamic_cast<QGIView*>(scene->parentItem());
         if (viewItem) {
-            TechDraw::DrawView* viewObj = viewItem->getViewObject();
-
             std::stringstream ss;
             if (edge) { ss << "Edge" << edge->getProjIndex(); }
             else if (vert) { ss << "Vertex" << vert->getProjIndex(); }
@@ -860,7 +858,7 @@ std::string MDIViewPage::getSceneSubName(QGraphicsItem* scene)
 }
 
 // adds scene to core selection if it's not in already.
-void MDIViewPage::addSceneToTreeSel(QGraphicsItem* sn, std::vector<Gui::SelectionObject> treeSel)
+void MDIViewPage::addSceneToTreeSel(QGraphicsItem* sn, [[maybe_unused]]std::vector<Gui::SelectionObject> treeSel)
 {
     auto* itemView = dynamic_cast<QGIView*>(sn);
     if (!itemView) {
@@ -955,7 +953,6 @@ void MDIViewPage::removeSelFromTreeSel(QList<QGraphicsItem*> sceneSel, Gui::Sele
                 TechDraw::DrawView* viewObj = itemView->getViewObject();
                 if (viewObj && !viewObj->isRemoving()) {
                     const char* doc_name = viewObj->getDocument()->getName();
-                    const char* obj_name = viewObj->getNameInDocument();
 
                     if (selDocName == doc_name && selObj == viewObj) {
                         found = true;
diff --git a/src/Mod/TechDraw/Gui/TechDrawHandler.cpp b/src/Mod/TechDraw/Gui/TechDrawHandler.cpp
index 182be5879b..bbcfd6cdf7 100644
--- a/src/Mod/TechDraw/Gui/TechDrawHandler.cpp
+++ b/src/Mod/TechDraw/Gui/TechDrawHandler.cpp
@@ -78,7 +78,7 @@ void TechDrawHandler::deactivate()
     // The context menu event of MDIViewPage comes after the tool is deactivated.
     // So to prevent the menu from appearing when the tool is cleared by right mouse click
     // we set a small timer.
-    QTimer::singleShot(100, [this]() { // 100 milliseconds delay
+    QTimer::singleShot(100, []() { // 100 milliseconds delay
         auto* mdi = dynamic_cast<MDIViewPage*>(Gui::getMainWindow()->activeWindow());
         if (!mdi) {
             return;