Replace raw pointers with unique_ptr

safer, simpler

1 files changed, 4 insertions, 10 deletions

diff --git a/src/client/QXmppTransferManager.cpp b/src/client/QXmppTransferManager.cpp
index 74e2e94f..b1f9be83 100644
--- a/src/client/QXmppTransferManager.cpp
+++ b/src/client/QXmppTransferManager.cpp
@@ -254,17 +254,14 @@ QXmppTransferJobPrivate::QXmppTransferJobPrivate()
 
 QXmppTransferJob::QXmppTransferJob(const QString &jid, QXmppTransferJob::Direction direction, QXmppClient *client, QObject *parent)
     : QXmppLoggable(parent),
-      d(new QXmppTransferJobPrivate)
+      d(std::make_unique<QXmppTransferJobPrivate>())
 {
     d->client = client;
     d->direction = direction;
     d->jid = jid;
 }
 
-QXmppTransferJob::~QXmppTransferJob()
-{
-    delete d;
-}
+QXmppTransferJob::~QXmppTransferJob() = default;
 
 ///
 /// Call this method if you wish to abort on ongoing transfer job.
@@ -760,7 +757,7 @@ QXmppTransferOutgoingJob *QXmppTransferManagerPrivate::getOutgoingJobByRequestId
 /// file transfers.
 ///
 QXmppTransferManager::QXmppTransferManager()
-    : d(new QXmppTransferManagerPrivate)
+    : d(std::make_unique<QXmppTransferManagerPrivate>())
 {
     // start SOCKS server
     d->socksServer = new QXmppSocksServer(this);
@@ -771,10 +768,7 @@ QXmppTransferManager::QXmppTransferManager()
     }
 }
 
-QXmppTransferManager::~QXmppTransferManager()
-{
-    delete d;
-}
+QXmppTransferManager::~QXmppTransferManager() = default;
 
 void QXmppTransferManager::byteStreamIqReceived(const QXmppByteStreamIq &iq)
 {