Apply 'modernize-make-unique' fix-it

Change-Id: I93783cfa3267f69237b0ad888d0747e4272cda0a
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>

9 files changed, 20 insertions, 12 deletions

diff --git a/src/lib/corelib/buildgraph/buildgraphloader.cpp b/src/lib/corelib/buildgraph/buildgraphloader.cpp
index bf6c30dcd..cad236db7 100644
--- a/src/lib/corelib/buildgraph/buildgraphloader.cpp
+++ b/src/lib/corelib/buildgraph/buildgraphloader.cpp
@@ -71,6 +71,7 @@
 
 #include <algorithm>
 #include <functional>
+#include <memory>
 #include <unordered_map>
 
 namespace qbs {
@@ -909,7 +910,7 @@ void BuildGraphLoader::rescueOldBuildData(const ResolvedProductConstPtr &restore
     if (!restoredProduct->buildData)
         return;
     if (!newlyResolvedProduct->buildData)
-        newlyResolvedProduct->buildData.reset(new ProductBuildData);
+        newlyResolvedProduct->buildData = std::make_unique<ProductBuildData>();
 
     qCDebug(lcBuildGraph) << "rescue data of product" << restoredProduct->uniqueName();
     QBS_CHECK(newlyResolvedProduct->buildData);
diff --git a/src/lib/corelib/buildgraph/projectbuilddata.cpp b/src/lib/corelib/buildgraph/projectbuilddata.cpp
index 31012e23e..0c7f7bfdc 100644
--- a/src/lib/corelib/buildgraph/projectbuilddata.cpp
+++ b/src/lib/corelib/buildgraph/projectbuilddata.cpp
@@ -60,6 +60,8 @@
 #include <tools/qttools.h>
 #include <tools/stlutils.h>
 
+#include <memory>
+
 namespace qbs {
 namespace Internal {
 
@@ -276,7 +278,7 @@ void BuildDataResolver::resolveBuildData(const TopLevelProjectPtr &resolvedProje
 {
     QBS_CHECK(!resolvedProject->buildData);
     m_project = resolvedProject;
-    resolvedProject->buildData.reset(new ProjectBuildData);
+    resolvedProject->buildData = std::make_unique<ProjectBuildData>();
     resolvedProject->buildData->evaluationContext = evalContext;
     const std::vector<ResolvedProductPtr> &allProducts = resolvedProject->allProducts();
     evalContext->initializeObserver(Tr::tr("Setting up build graph for configuration %1")
@@ -390,7 +392,7 @@ void BuildDataResolver::resolveProductBuildData(const ResolvedProductPtr &produc
 
     evalContext()->checkForCancelation();
 
-    product->buildData.reset(new ProductBuildData);
+    product->buildData = std::make_unique<ProductBuildData>();
     ArtifactSetByFileTag artifactsPerFileTag;
 
     for (const auto &dependency : qAsConst(product->dependencies)) {
diff --git a/src/lib/corelib/generators/xmlprojectwriter.cpp b/src/lib/corelib/generators/xmlprojectwriter.cpp
index 5554e5935..190e1304f 100644
--- a/src/lib/corelib/generators/xmlprojectwriter.cpp
+++ b/src/lib/corelib/generators/xmlprojectwriter.cpp
@@ -33,6 +33,7 @@
 #include "xmlproperty.h"
 #include "xmlpropertygroup.h"
 
+#include <memory>
 #include <ostream>
 
 namespace qbs {
@@ -42,7 +43,7 @@ namespace xml {
 ProjectWriter::ProjectWriter(std::ostream *device)
     : m_device(device)
 {
-    m_writer.reset(new QXmlStreamWriter(&m_buffer));
+    m_writer = std::make_unique<QXmlStreamWriter>(&m_buffer);
     m_writer->setAutoFormatting(true);
 }
 
diff --git a/src/lib/corelib/generators/xmlworkspacewriter.cpp b/src/lib/corelib/generators/xmlworkspacewriter.cpp
index c88cb06d0..d21b63c42 100644
--- a/src/lib/corelib/generators/xmlworkspacewriter.cpp
+++ b/src/lib/corelib/generators/xmlworkspacewriter.cpp
@@ -33,6 +33,7 @@
 #include "xmlworkspace.h"
 #include "xmlworkspacewriter.h"
 
+#include <memory>
 #include <ostream>
 
 namespace qbs {
@@ -42,7 +43,7 @@ namespace xml {
 WorkspaceWriter::WorkspaceWriter(std::ostream *device)
     : m_device(device)
 {
-    m_writer.reset(new QXmlStreamWriter(&m_buffer));
+    m_writer = std::make_unique<QXmlStreamWriter>(&m_buffer);
     m_writer->setAutoFormatting(true);
 }
 
diff --git a/src/lib/corelib/language/language.cpp b/src/lib/corelib/language/language.cpp
index 53715fd57..8608e08c7 100644
--- a/src/lib/corelib/language/language.cpp
+++ b/src/lib/corelib/language/language.cpp
@@ -356,7 +356,7 @@ FileTags ResolvedProduct::fileTagsForFileName(const QString &fileName) const
                         return result;
                     }
                 } else {
-                    priority.reset(new int(tagger->priority()));
+                    priority = std::make_unique<int>(tagger->priority());
                 }
                 result.unite(tagger->fileTags());
                 break;
diff --git a/src/lib/corelib/language/moduleloader.cpp b/src/lib/corelib/language/moduleloader.cpp
index 5568febbd..1aeca0411 100644
--- a/src/lib/corelib/language/moduleloader.cpp
+++ b/src/lib/corelib/language/moduleloader.cpp
@@ -79,6 +79,7 @@
 #include <QtScript/qscriptvalueiterator.h>
 
 #include <algorithm>
+#include <memory>
 #include <utility>
 
 namespace qbs {
@@ -292,7 +293,7 @@ ModuleLoaderResult ModuleLoader::load(const SetupProjectParameters &parameters)
             = m_elapsedTimePropertyChecking = 0;
     m_elapsedTimeProbes = 0;
     m_probesEncountered = m_probesRun = m_probesCachedCurrent = m_probesCachedOld = 0;
-    m_settings.reset(new Settings(parameters.settingsDirectory()));
+    m_settings = std::make_unique<Settings>(parameters.settingsDirectory());
 
     const auto keys = m_parameters.overriddenValues().keys();
     for (const QString &key : keys) {
diff --git a/src/lib/corelib/language/projectresolver.cpp b/src/lib/corelib/language/projectresolver.cpp
index 868ad32af..6b678cb8f 100644
--- a/src/lib/corelib/language/projectresolver.cpp
+++ b/src/lib/corelib/language/projectresolver.cpp
@@ -71,6 +71,7 @@
 #include <QtCore/qregexp.h>
 
 #include <algorithm>
+#include <memory>
 #include <queue>
 
 namespace qbs {
@@ -810,7 +811,7 @@ void ProjectResolver::resolveGroupFully(Item *item, ProjectResolver::ProjectCont
         group->targetOfModule = moduleProp->value().toString();
     ErrorInfo fileError;
     if (!patterns.empty()) {
-        group->wildcards = std::unique_ptr<SourceWildCards>(new SourceWildCards);
+        group->wildcards = std::make_unique<SourceWildCards>();
         SourceWildCards *wildcards = group->wildcards.get();
         wildcards->group = group.get();
         wildcards->excludePatterns = m_evaluator->stringListValue(
diff --git a/src/lib/corelib/tools/settingscreator.cpp b/src/lib/corelib/tools/settingscreator.cpp
index eaaf2802c..b7c727cd0 100644
--- a/src/lib/corelib/tools/settingscreator.cpp
+++ b/src/lib/corelib/tools/settingscreator.cpp
@@ -1,3 +1,4 @@
+#include <memory>
 #include <utility>
 
 /****************************************************************************
@@ -104,7 +105,7 @@ void SettingsCreator::migrate()
     // Adapt all paths in settings that point to the old location. At the time of this writing,
     // that's only preferences.qbsSearchPaths as written by libqtprofilesetup, but we don't want
     // to hardcode that here.
-    m_settings.reset(new QSettings(m_newSettingsFilePath, format()));
+    m_settings = std::make_unique<QSettings>(m_newSettingsFilePath, format());
     const auto allKeys = m_settings->allKeys();
     for (const QString &key : allKeys) {
         QVariant v = m_settings->value(key);
@@ -136,7 +137,7 @@ void SettingsCreator::createQSettings()
     m_newSettingsDir = m_settingsBaseDir + QLatin1String("/qbs/") + m_qbsVersion.toString();
     m_settingsFileName = fi.fileName();
     m_newSettingsFilePath = m_newSettingsDir + QLatin1Char('/') + m_settingsFileName;
-    m_settings.reset(new QSettings(m_newSettingsFilePath, tmp->format()));
+    m_settings = std::make_unique<QSettings>(m_newSettingsFilePath, tmp->format());
 }
 
 Version SettingsCreator::predecessor() const
diff --git a/src/lib/corelib/tools/settingsmodel.cpp b/src/lib/corelib/tools/settingsmodel.cpp
index d67f1da5c..7283e959c 100644
--- a/src/lib/corelib/tools/settingsmodel.cpp
+++ b/src/lib/corelib/tools/settingsmodel.cpp
@@ -125,7 +125,7 @@ public:
 SettingsModel::SettingsModel(const QString &settingsDir, Settings::Scope scope, QObject *parent)
     : QAbstractItemModel(parent), d(new SettingsModelPrivate)
 {
-    d->settings.reset(new qbs::Settings(settingsDir));
+    d->settings = std::make_unique<qbs::Settings>(settingsDir);
     d->settings->setScopeForWriting(scope);
     d->readSettings();
 }
@@ -155,7 +155,7 @@ void SettingsModel::updateSettingsDir(const QString &settingsDir)
 {
     const Settings::Scope scope = d->scope();
     beginResetModel();
-    d->settings.reset(new qbs::Settings(settingsDir));
+    d->settings = std::make_unique<qbs::Settings>(settingsDir);
     d->settings->setScopeForWriting(scope);
     d->readSettings();
     endResetModel();