not necessary to use QPointer here

5ecb9599 · Laurent Montel · 315d4ba1 · 5ecb9599
Commit 5ecb9599 authored Oct 06, 2020 by Laurent Montel 😁
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 3 deletions

kmail/editorinitplugins/externalcomposer/externalcomposerplugineditor.cpp ...plugins/externalcomposer/externalcomposerplugineditor.cpp +2 -3

No files found.
--- a/kmail/editorinitplugins/externalcomposer/externalcomposerplugineditor.cpp
+++ b/kmail/editorinitplugins/externalcomposer/externalcomposerplugineditor.cpp
@@ -33,9 +33,8 @@ bool ExternalComposerPluginEditor::hasConfigureDialog() const

 void ExternalComposerPluginEditor::showConfigureDialog(QWidget *parent)
 {
-    QPointer<ExternalComposerConfigureDialog> dlg = new ExternalComposerConfigureDialog(parent);
-    dlg->exec();
-    delete dlg;
+    ExternalComposerConfigureDialog dlg(parent);
+    dlg.exec();
 }

 #include "externalcomposerplugineditor.moc"