private pointer now scoped (cleaner)

Author: Skycoder42

AutoUpdaterCore/updater.cpp

@@ -23,10 +23,7 @@ Updater::Updater(const QString &maintenanceToolPath, QObject *parent) :
 	d->toolPath = UpdaterPrivate::toSystemExe(maintenanceToolPath);
 }
 
-Updater::~Updater()
-{
-	delete this->d_ptr;
-}
+Updater::~Updater() {}
 
 bool Updater::exitedNormally() const
 {

AutoUpdaterCore/updater.h

@@ -8,6 +8,7 @@
 #include <QDateTime>
 #include <QVersionNumber>
 #include <QLoggingCategory>
+#include <QScopedPointer>
 #include "adminauthoriser.h"
 
 namespace QtAutoUpdater
@@ -100,7 +101,7 @@ namespace QtAutoUpdater
 		void updateInfoChanged(QList<QtAutoUpdater::Updater::UpdateInfo> updateInfo);
 
 	private:
-		UpdaterPrivate *d_ptr;
+		QScopedPointer<UpdaterPrivate> d_ptr;
 		Q_DECLARE_PRIVATE(Updater)
 	};
 }

AutoUpdaterGui/updatebutton.cpp

@@ -10,10 +10,7 @@ UpdateButton::UpdateButton(QWidget *parent, UpdateController *controller) :
 	d_ptr(new UpdateButtonPrivate(this, controller))
 {}
 
-UpdateButton::~UpdateButton()
-{
-	delete this->d_ptr;
-}
+UpdateButton::~UpdateButton(){}
 
 QString UpdateButton::animationFile() const
 {

AutoUpdaterGui/updatebutton.h

@@ -2,6 +2,7 @@
 #define UPDATEBUTTON_H
 
 #include <QWidget>
+#include <QScopedPointer>
 #include "updatecontroller.h"
 
 namespace QtAutoUpdater
@@ -65,7 +66,7 @@ namespace QtAutoUpdater
 		void controllerDestroyed();
 
 	private:
-		UpdateButtonPrivate *d_ptr;
+		QScopedPointer<UpdateButtonPrivate> d_ptr;
 		Q_DECLARE_PRIVATE(UpdateButton)
 		UpdateController* m_controller;
 	};

AutoUpdaterGui/updatebutton_p.h

@@ -13,6 +13,7 @@ namespace QtAutoUpdater
 {
 	class UpdateButtonPrivate
 	{
+	public:
 		UpdateButton *q_ptr;
 		Q_DECLARE_PUBLIC(UpdateButton)
 

AutoUpdaterGui/updatecontroller.cpp

@@ -35,10 +35,7 @@ UpdateController::UpdateController(const QString &maintenanceToolPath, QWidget *
 	d_ptr(new UpdateControllerPrivate(this, maintenanceToolPath, parentWidget))
 {}
 
-UpdateController::~UpdateController()
-{
-	delete this->d_ptr;
-}
+UpdateController::~UpdateController(){}
 
 QAction *UpdateController::createUpdateAction(QObject *parent)
 {

AutoUpdaterGui/updatecontroller.h

@@ -4,6 +4,7 @@
 #include <QObject>
 #include <QAction>
 #include <QStringList>
+#include <QScopedPointer>
 
 namespace QtAutoUpdater
 {
@@ -108,7 +109,7 @@ namespace QtAutoUpdater
 		void timerTriggered(const QVariant &parameter);
 
 	private:
-		UpdateControllerPrivate *d_ptr;
+		QScopedPointer<UpdateControllerPrivate> d_ptr;
 		Q_DECLARE_PRIVATE(UpdateController)
 	};
 }

AutoUpdaterGui/updatecontroller_p.h

@@ -12,6 +12,7 @@ namespace QtAutoUpdater
 {
 	class UpdateControllerPrivate//TODO load icon indirectly -> easier replacement
 	{
+	public:
 		typedef QPair<UpdateController::DisplayLevel, bool> UpdateTask;
 
 		UpdateController *q_ptr;