QmlProfiler: Use QPointer for storing config widget

Most other options pages do it this way and none of them delete the
widget in the destructor. Apparently we can rely on finish() getting
called in time.

Change-Id: Ie0964d291bc8ab3eea537921bf476012d58776a4
Reviewed-by: Alessandro Portale <[email protected]>

1 files changed, 1 insertions, 7 deletions

diff --git a/src/plugins/qmlprofiler/qmlprofileroptionspage.cpp b/src/plugins/qmlprofiler/qmlprofileroptionspage.cpp
index ec433997d93..dda136d68b2 100644
--- a/src/plugins/qmlprofiler/qmlprofileroptionspage.cpp
+++ b/src/plugins/qmlprofiler/qmlprofileroptionspage.cpp
@@ -36,7 +36,7 @@
 namespace QmlProfiler {
 namespace Internal {
 
-QmlProfilerOptionsPage::QmlProfilerOptionsPage() : m_widget(0)
+QmlProfilerOptionsPage::QmlProfilerOptionsPage()
 {
     setId(Constants::SETTINGS);
     setDisplayName(tr("QML Profiler"));
@@ -45,11 +45,6 @@ QmlProfilerOptionsPage::QmlProfilerOptionsPage() : m_widget(0)
     setCategoryIcon(QLatin1String(":/images/analyzer_category.png"));
 }
 
-QmlProfilerOptionsPage::~QmlProfilerOptionsPage()
-{
-    delete m_widget;
-}
-
 QWidget *QmlProfilerOptionsPage::widget()
 {
     // We cannot parent the widget to the options page as it expects a QWidget as parent
@@ -66,7 +61,6 @@ void QmlProfilerOptionsPage::apply()
 void QmlProfilerOptionsPage::finish()
 {
     delete m_widget;
-    m_widget = 0;
 }
 
 } // Internal