create modal dialogs on stack as David suggested

svn path=/trunk/KDE/kdebase/apps/; revision=668900

diff --git a/src/dolphincontextmenu.cpp b/src/dolphincontextmenu.cpp
index 4e2cb4f8629721cf66a9b78a998be5d5ffb9a154..468b7d69d0f5dfe049df3fdd14943493d9edcc93 100644 (file)
--- a/src/dolphincontextmenu.cpp
+++ b/src/dolphincontextmenu.cpp
@@ -243,9 +243,8 @@ void DolphinContextMenu::openViewportContextMenu()
     QAction* activatedAction = popup->exec(QCursor::pos());
     if (activatedAction == propertiesAction) {
         const KUrl& url = m_mainWindow->activeView()->url();
-        KPropertiesDialog* dlg = new KPropertiesDialog(url);
-        dlg->exec();
-        delete dlg;
+        KPropertiesDialog dialog(url);
+        dialog.exec();
     } else if (activatedAction == bookmarkAction) {
         const KUrl& url = m_mainWindow->activeView()->url();
         if (url.isValid()) {

diff --git a/src/dolphinmainwindow.cpp b/src/dolphinmainwindow.cpp
index cdb3f3cbca944f063a34a4b55d612e2c82b48793..59fc84af33dc4e3043a1a84906538037187ef82d 100644 (file)
--- a/src/dolphinmainwindow.cpp
+++ b/src/dolphinmainwindow.cpp
@@ -458,9 +458,8 @@ void DolphinMainWindow::deleteItems()
 void DolphinMainWindow::properties()
 {
     const KFileItemList list = m_activeView->selectedItems();
-    KPropertiesDialog* dlg = new KPropertiesDialog(list, this);
-    dlg->exec();
-    delete dlg;
+    KPropertiesDialog dialog(list, this);
+    dialog.exec();
 }
 
 void DolphinMainWindow::quit()

diff --git a/src/treeviewcontextmenu.cpp b/src/treeviewcontextmenu.cpp
index fb6db1764cbd9de0b065efa1d017a5354c6ddd88..ec6d322008b5e98e66f14e2f86e6fd16111cfdb4 100644 (file)
--- a/src/treeviewcontextmenu.cpp
+++ b/src/treeviewcontextmenu.cpp
@@ -162,9 +162,8 @@ void TreeViewContextMenu::deleteItem()
 
 void TreeViewContextMenu::showProperties()
 {
-    KPropertiesDialog* dlg = new KPropertiesDialog(m_fileInfo->url() );
-    dlg->exec();
-    delete dlg;
+    KPropertiesDialog dialog(m_fileInfo->url());
+    dialog.exec();
 }
 
 #include "treeviewcontextmenu.moc"