Merge pull request #8599 from m-kuhn/auth-manager-unique-ptr

m-kuhn · web-flow · commit 774c890f9cdf · 2018-12-04T12:09:03.000+01:00
Use std::unique_ptr
diff --git a/src/core/auth/qgsauthmanager.cpp b/src/core/auth/qgsauthmanager.cpp
@@ -175,7 +175,7 @@ bool QgsAuthManager::init( const QString &pluginPath, const QString &authDatabas
   mAuthInit = true;
 
   QgsDebugMsg( QStringLiteral( "Initializing QCA..." ) );
-  mQcaInitializer = new QCA::Initializer( QCA::Practical, 256 );
+  mQcaInitializer = qgis::make_unique<QCA::Initializer>( QCA::Practical, 256 );
 
   QgsDebugMsg( QStringLiteral( "QCA initialized." ) );
   QCA::scanForPlugins();
@@ -2974,8 +2974,6 @@ QgsAuthManager::~QgsAuthManager()
   mMutex = nullptr;
   delete mScheduledDbEraseTimer;
   mScheduledDbEraseTimer = nullptr;
-  delete mQcaInitializer;
-  mQcaInitializer = nullptr;
   QSqlDatabase::removeDatabase( QStringLiteral( "authentication.configs" ) );
 }
 
diff --git a/src/core/auth/qgsauthmanager.h b/src/core/auth/qgsauthmanager.h
@@ -848,7 +848,7 @@ class CORE_EXPORT QgsAuthManager : public QObject
     bool mAuthInit = false;
     QString mAuthDbPath;
 
-    QCA::Initializer *mQcaInitializer = nullptr;
+    std::unique_ptr<QCA::Initializer> mQcaInitializer;
 
     QHash<QString, QString> mConfigAuthMethods;
     QHash<QString, QgsAuthMethod *> mAuthMethods;
