From 435339f2ae6ccd88df1ac44958b0b1736f77386a Mon Sep 17 00:00:00 2001
From: Tobias Hunger <tobias.hunger@nokia.com>
Date: Wed, 24 Feb 2010 17:52:56 +0100
Subject: [PATCH] Ask before removing targets

Reviewed-by: Robert Loehning
---
 src/plugins/projectexplorer/targetsettingspanel.cpp | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/plugins/projectexplorer/targetsettingspanel.cpp b/src/plugins/projectexplorer/targetsettingspanel.cpp
index d9e61a8dce3..d0cf74e873f 100644
--- a/src/plugins/projectexplorer/targetsettingspanel.cpp
+++ b/src/plugins/projectexplorer/targetsettingspanel.cpp
@@ -41,6 +41,7 @@
 
 #include <QtCore/QCoreApplication>
 #include <QtGui/QLabel>
+#include <QtGui/QMessageBox>
 #include <QtGui/QVBoxLayout>
 
 using namespace ProjectExplorer;
@@ -249,8 +250,13 @@ void TargetSettingsPanelWidget::removeTarget()
 {
     int index = m_selector->currentIndex();
     Target *t = m_targets.at(index);
-    // TODO: Ask before removal?
-    m_project->removeTarget(t);
+    int ret = QMessageBox::warning(this, tr("Qt Creator"),
+                                   tr("Do you really want to remove the\n"
+                                      "\"%1\" target?").arg(t->displayName()),
+                                    QMessageBox::Yes | QMessageBox::No,
+                                    QMessageBox::No);
+    if (ret == QMessageBox::Yes)
+        m_project->removeTarget(t);
 }
 
 void TargetSettingsPanelWidget::targetAdded(ProjectExplorer::Target *target)
-- 
GitLab