Merge remote-tracking branch 'origin/9.0'

Change-Id: I2a2635e449b29f4a1c7cb8a9359ba9184357285a

21 files changed, 177 insertions, 396 deletions

diff --git a/.github/workflows/build_cmake.yml b/.github/workflows/build_cmake.yml
index c2b46ae..97dba40 100644
--- a/.github/workflows/build_cmake.yml
+++ b/.github/workflows/build_cmake.yml
@@ -4,8 +4,8 @@ on: [push]
 
 env:
   PLUGIN_NAME: Fossil
-  QT_VERSION: 6.3.1
-  QT_CREATOR_VERSION: 8.0.0-beta2
+  QT_VERSION: 6.4.0
+  QT_CREATOR_VERSION: 9.0.0-beta1
   QT_CREATOR_SNAPSHOT: latest
   CMAKE_VERSION: 3.21.1
   NINJA_VERSION: 1.10.2
diff --git a/plugins/fossil/CMakeLists.txt b/plugins/fossil/CMakeLists.txt
index ad94bb9..0b58b16 100644
--- a/plugins/fossil/CMakeLists.txt
+++ b/plugins/fossil/CMakeLists.txt
@@ -12,7 +12,7 @@ add_qtc_plugin(Fossil
   DEPENDS Qt5::Widgets QtCreator::ExtensionSystem QtCreator::Utils
   SOURCES
     annotationhighlighter.cpp annotationhighlighter.h
-    branchinfo.cpp branchinfo.h
+    branchinfo.h
     commiteditor.cpp commiteditor.h
     configuredialog.cpp configuredialog.h configuredialog.ui
     constants.h
@@ -25,6 +25,6 @@ add_qtc_plugin(Fossil
     fossilsettings.cpp fossilsettings.h
     pullorpushdialog.cpp pullorpushdialog.h pullorpushdialog.ui
     revertdialog.ui
-    revisioninfo.cpp revisioninfo.h
+    revisioninfo.h
     wizard/fossiljsextension.cpp wizard/fossiljsextension.h
 )
diff --git a/plugins/fossil/annotationhighlighter.cpp b/plugins/fossil/annotationhighlighter.cpp
index a910bb7..92aee09 100644
--- a/plugins/fossil/annotationhighlighter.cpp
+++ b/plugins/fossil/annotationhighlighter.cpp
@@ -44,7 +44,7 @@ QString FossilAnnotationHighlighter::changeNumber(const QString &block) const
     QRegularExpressionMatch changesetIdMatch = m_changesetIdPattern.match(block);
     if (changesetIdMatch.hasMatch())
         return changesetIdMatch.captured(1);
-    return QString();
+    return {};
 }
 
 } // namespace Internal
diff --git a/plugins/fossil/branchinfo.cpp b/plugins/fossil/branchinfo.cpp
deleted file mode 100644
index 08cdd26..0000000
--- a/plugins/fossil/branchinfo.cpp
+++ /dev/null
@@ -1,57 +0,0 @@
-/****************************************************************************
-**
-** Copyright (c) 2018 Artur Shepilko
-** Contact: https://www.qt.io/licensing/
-**
-** This file is part of Qt Creator.
-**
-** Commercial License Usage
-** Licensees holding valid commercial Qt licenses may use this file in
-** accordance with the commercial license agreement provided with the
-** Software or, alternatively, in accordance with the terms contained in
-** a written agreement between you and The Qt Company. For licensing terms
-** and conditions see https://www.qt.io/terms-conditions. For further
-** information use the contact form at https://www.qt.io/contact-us.
-**
-** GNU General Public License Usage
-** Alternatively, this file may be used under the terms of the GNU
-** General Public License version 3 as published by the Free Software
-** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT
-** included in the packaging of this file. Please review the following
-** information to ensure the GNU General Public License requirements will
-** be met: https://www.gnu.org/licenses/gpl-3.0.html.
-**
-****************************************************************************/
-
-#include "branchinfo.h"
-
-namespace Fossil {
-namespace Internal {
-
-BranchInfo::BranchInfo(const QString &name, BranchFlags flags) :
-    m_name(name),
-    m_flags(flags)
-{ }
-
-const QString &BranchInfo::name() const
-{
-    return m_name;
-}
-
-bool BranchInfo::isCurrent() const
-{
-    return m_flags.testFlag(Current);
-}
-
-bool BranchInfo::isClosed() const
-{
-    return m_flags.testFlag(Closed);
-}
-
-bool BranchInfo::isPrivate() const
-{
-    return m_flags.testFlag(Private);
-}
-
-} // namespace Internal
-} // namespace Fossil
diff --git a/plugins/fossil/branchinfo.h b/plugins/fossil/branchinfo.h
index 034280b..336ea9e 100644
--- a/plugins/fossil/branchinfo.h
+++ b/plugins/fossil/branchinfo.h
@@ -40,17 +40,10 @@ public:
     };
     Q_DECLARE_FLAGS(BranchFlags, BranchFlag)
 
-    explicit BranchInfo(const QString &name = QString(), BranchFlags flags = {});
+    bool isCurrent() const { return flags.testFlag(Current); }
 
-public:
-    const QString &name() const;
-    bool isCurrent() const;
-    bool isClosed() const;
-    bool isPrivate() const;
-
-private:
-    QString m_name;
-    BranchFlags m_flags;
+    QString name;
+    BranchFlags flags;
 };
 
 Q_DECLARE_OPERATORS_FOR_FLAGS(BranchInfo::BranchFlags)
diff --git a/plugins/fossil/commiteditor.cpp b/plugins/fossil/commiteditor.cpp
index 58d6214..4ed3370 100644
--- a/plugins/fossil/commiteditor.cpp
+++ b/plugins/fossil/commiteditor.cpp
@@ -58,9 +58,7 @@ void CommitEditor::setFields(const QString &repositoryRoot, const BranchInfo &br
 
     m_fileModel = new VcsBase::SubmitFileModel(this);
     m_fileModel->setRepositoryRoot(repositoryRoot);
-    m_fileModel->setFileStatusQualifier([](const QString &status, const QVariant &)
-                                           -> VcsBase::SubmitFileModel::FileStatusHint
-    {
+    m_fileModel->setFileStatusQualifier([](const QString &status, const QVariant &) {
         if (status == Constants::FSTATUS_ADDED
             || status == Constants::FSTATUS_ADDED_BY_MERGE
             || status == Constants::FSTATUS_ADDED_BY_INTEGRATE) {
@@ -75,12 +73,13 @@ void CommitEditor::setFields(const QString &repositoryRoot, const BranchInfo &br
             return VcsBase::SubmitFileModel::FileRenamed;
         }
         return VcsBase::SubmitFileModel::FileStatusUnknown;
-    } );
+    });
 
     const QList<VcsBase::VcsBaseClient::StatusItem> toAdd = Utils::filtered(repoStatus,
-                [](const VcsBase::VcsBaseClient::StatusItem &item) { return item.flags != Constants::FSTATUS_UNKNOWN; });
+                [](const VcsBase::VcsBaseClient::StatusItem &item)
+    { return item.flags != Constants::FSTATUS_UNKNOWN; });
     for (const VcsBase::VcsBaseClient::StatusItem &item : toAdd)
-            m_fileModel->addFile(item.file, item.flags);
+        m_fileModel->addFile(item.file, item.flags);
 
     setFileModel(m_fileModel);
 }
diff --git a/plugins/fossil/configuredialog.cpp b/plugins/fossil/configuredialog.cpp
index a6e72b9..a19ae5d 100644
--- a/plugins/fossil/configuredialog.cpp
+++ b/plugins/fossil/configuredialog.cpp
@@ -37,13 +37,12 @@ namespace Internal {
 
 class ConfigureDialogPrivate {
 public:
-    RepositorySettings settings() {
-        m_settings.user = m_ui.userLineEdit->text().trimmed();
-        m_settings.sslIdentityFile = m_ui.sslIdentityFilePathChooser->filePath().toString();
-        m_settings.autosync =
-                (m_ui.disableAutosyncCheckBox->isChecked() ? RepositorySettings::AutosyncOff
-                                                           : RepositorySettings::AutosyncOn);
-        return m_settings;
+    RepositorySettings settings() const
+    {
+        return {m_ui.userLineEdit->text().trimmed(),
+                m_ui.sslIdentityFilePathChooser->filePath().toString(),
+                m_ui.disableAutosyncCheckBox->isChecked()
+                    ? RepositorySettings::AutosyncOff : RepositorySettings::AutosyncOn};
     }
 
     void updateUi() {
diff --git a/plugins/fossil/fossil.qbs b/plugins/fossil/fossil.qbs
index e89b691..0bab349 100644
--- a/plugins/fossil/fossil.qbs
+++ b/plugins/fossil/fossil.qbs
@@ -21,9 +21,9 @@ QtcPlugin {
         "fossileditor.cpp", "fossileditor.h",
         "annotationhighlighter.cpp", "annotationhighlighter.h",
         "pullorpushdialog.cpp", "pullorpushdialog.h", "pullorpushdialog.ui",
-        "branchinfo.cpp", "branchinfo.h",
+        "branchinfo.h",
         "configuredialog.cpp", "configuredialog.h", "configuredialog.ui",
-        "revisioninfo.cpp", "revisioninfo.h",
+        "revisioninfo.h",
         "fossil.qrc",
         "revertdialog.ui",
         "fossilcommitpanel.ui",
diff --git a/plugins/fossil/fossilclient.cpp b/plugins/fossil/fossilclient.cpp
index 02306a3..f6bd4a6 100644
--- a/plugins/fossil/fossilclient.cpp
+++ b/plugins/fossil/fossilclient.cpp
@@ -57,8 +57,7 @@ using namespace VcsBase;
 namespace Fossil {
 namespace Internal {
 
-const unsigned s_pullFlags = VcsCommand::ShowStdOut | VcsCommand::ShowSuccessMessage;
-
+const RunFlags s_pullFlags = RunFlags::ShowStdOut | RunFlags::ShowSuccessMessage;
 
 // Parameter widget controlling whitespace diff mode, associated with a parameter
 class FossilDiffConfig : public VcsBaseEditorConfig
@@ -122,7 +121,6 @@ public:
         QTC_ASSERT(client, return);
         addReloadButton();
     }
-
 };
 
 class FossilLogConfig : public VcsBaseEditorConfig
@@ -199,7 +197,8 @@ public:
         QStringList args;
 
         // split "-t val" => "-t" "val"
-        foreach (const QString &arg, VcsBaseEditorConfig::arguments()) {
+        const QStringList arguments = VcsBaseEditorConfig::arguments();
+        for (const QString &arg : arguments) {
             if (arg.startsWith("-t")) {
                 args << arg.split(' ');
 
@@ -214,12 +213,10 @@ public:
                     }
                     args << params[i];
                 }
-
             } else {
                 args << arg;
             }
         }
-
         return args;
     }
 
@@ -265,9 +262,7 @@ unsigned int FossilClient::synchronousBinaryVersion() const
     if (settings().binaryPath.value().isEmpty())
         return 0;
 
-    QStringList args("version");
-
-    const CommandResult result = vcsSynchronousExec(FilePath(), args);
+    const CommandResult result = vcsSynchronousExec({}, QStringList{"version"});
     if (result.result() != ProcessResult::FinishedWithSuccess)
         return 0;
 
@@ -282,31 +277,34 @@ unsigned int FossilClient::synchronousBinaryVersion() const
     const int major = versionMatch.captured(1).toInt();
     const int minor = versionMatch.captured(2).toInt();
     const int patch = 0;
-    return makeVersionNumber(major,minor,patch);
+    return makeVersionNumber(major, minor, patch);
 }
 
-QList<BranchInfo> FossilClient::branchListFromOutput(const QString &output, const BranchInfo::BranchFlags defaultFlags)
+QList<BranchInfo> FossilClient::branchListFromOutput(const QString &output,
+                                                     const BranchInfo::BranchFlags defaultFlags)
 {
     // Branch list format:
     // "  branch-name"
     // "* current-branch"
-    return Utils::transform(output.split('\n', Qt::SkipEmptyParts), [=](const QString &l) {
+    return Utils::transform(output.split('\n', Qt::SkipEmptyParts),
+                            [=](const QString &l) -> BranchInfo {
         const QString &name = l.mid(2);
-        QTC_ASSERT(!name.isEmpty(), return BranchInfo());
-        const BranchInfo::BranchFlags flags = (l.startsWith("* ") ? defaultFlags | BranchInfo::Current : defaultFlags);
-        return BranchInfo(name, flags);
+        QTC_ASSERT(!name.isEmpty(), return {});
+        const BranchInfo::BranchFlags flags = (l.startsWith("* ")
+                                               ? defaultFlags | BranchInfo::Current : defaultFlags);
+        return {name, flags};
     });
 }
 
 BranchInfo FossilClient::synchronousCurrentBranch(const FilePath &workingDirectory)
 {
     if (workingDirectory.isEmpty())
-        return BranchInfo();
+        return {};
 
     // First try to get the current branch from the list of open branches
     const CommandResult result = vcsSynchronousExec(workingDirectory, {"branch", "list"});
     if (result.result() != ProcessResult::FinishedWithSuccess)
-        return BranchInfo();
+        return {};
 
     const QString output = sanitizeFossilOutput(result.cleanedStdOut());
     BranchInfo currentBranch = Utils::findOrDefault(branchListFromOutput(output), [](const BranchInfo &b) {
@@ -315,9 +313,10 @@ BranchInfo FossilClient::synchronousCurrentBranch(const FilePath &workingDirecto
 
     if (!currentBranch.isCurrent()) {
         // If not available from open branches, request it from the list of closed branches.
-        const CommandResult result = vcsSynchronousExec(workingDirectory, {"branch", "list", "--closed"});
+        const CommandResult result = vcsSynchronousExec(workingDirectory,
+                                                        {"branch", "list", "--closed"});
         if (result.result() != ProcessResult::FinishedWithSuccess)
-            return BranchInfo();
+            return {};
 
         const QString output = sanitizeFossilOutput(result.cleanedStdOut());
         currentBranch = Utils::findOrDefault(branchListFromOutput(output, BranchInfo::Closed), [](const BranchInfo &b) {
@@ -334,12 +333,12 @@ QList<BranchInfo> FossilClient::synchronousBranchQuery(const FilePath &workingDi
     // Sort the list by branch name.
 
     if (workingDirectory.isEmpty())
-        return QList<BranchInfo>();
+        return {};
 
     // First get list of open branches
     CommandResult result = vcsSynchronousExec(workingDirectory, {"branch", "list"});
     if (result.result() != ProcessResult::FinishedWithSuccess)
-        return QList<BranchInfo>();
+        return {};
 
     QString output = sanitizeFossilOutput(result.cleanedStdOut());
     QList<BranchInfo> branches = branchListFromOutput(output);
@@ -347,13 +346,13 @@ QList<BranchInfo> FossilClient::synchronousBranchQuery(const FilePath &workingDi
     // Append a list of closed branches.
     result = vcsSynchronousExec(workingDirectory, {"branch", "list", "--closed"});
     if (result.result() != ProcessResult::FinishedWithSuccess)
-        return QList<BranchInfo>();
+        return {};
 
     output = sanitizeFossilOutput(result.cleanedStdOut());
     branches.append(branchListFromOutput(output, BranchInfo::Closed));
 
     std::sort(branches.begin(), branches.end(),
-          [](const BranchInfo &a, const BranchInfo &b) { return a.name() < b.name(); });
+          [](const BranchInfo &a, const BranchInfo &b) { return a.name < b.name; });
     return branches;
 }
 
@@ -363,13 +362,13 @@ QStringList FossilClient::parseRevisionCommentLine(const QString &commentLine)
 
     const QRegularExpression commentRx("^comment:\\s+(.*)\\s\\(user:\\s(.*)\\)$",
                                        QRegularExpression::CaseInsensitiveOption);
-    QTC_ASSERT(commentRx.isValid(), return QStringList());
+    QTC_ASSERT(commentRx.isValid(), return {});
 
     const QRegularExpressionMatch match = commentRx.match(commentLine);
     if (!match.hasMatch())
-        return QStringList();
+        return {};
 
-    return QStringList({match.captured(1), match.captured(2)});
+    return {match.captured(1), match.captured(2)};
 }
 
 RevisionInfo FossilClient::synchronousRevisionQuery(const FilePath &workingDirectory,
@@ -379,16 +378,16 @@ RevisionInfo FossilClient::synchronousRevisionQuery(const FilePath &workingDirec
     // Query details of the given revision/check-out id,
     // if none specified, provide information about current revision
     if (workingDirectory.isEmpty())
-        return RevisionInfo();
+        return {};
 
     QStringList args("info");
     if (!id.isEmpty())
         args << id;
 
     const CommandResult result = vcsSynchronousExec(workingDirectory, args,
-                                                         VcsCommand::SuppressCommandLogging);
+                                                    RunFlags::SuppressCommandLogging);
     if (result.result() != ProcessResult::FinishedWithSuccess)
-        return RevisionInfo();
+        return {};
 
     const QString output = sanitizeFossilOutput(result.cleanedStdOut());
 
@@ -399,7 +398,7 @@ RevisionInfo FossilClient::synchronousRevisionQuery(const FilePath &workingDirec
     QString committer;
 
     const QRegularExpression idRx("([0-9a-f]{5,40})");
-    QTC_ASSERT(idRx.isValid(), return RevisionInfo());
+    QTC_ASSERT(idRx.isValid(), return {});
 
     const QString hashToken =
             QString::fromUtf8(supportedFeatures().testFlag(InfoHashFeature) ? "hash: " : "uuid: ");
@@ -408,7 +407,7 @@ RevisionInfo FossilClient::synchronousRevisionQuery(const FilePath &workingDirec
         if (l.startsWith("checkout: ", Qt::CaseInsensitive)
             || l.startsWith(hashToken, Qt::CaseInsensitive)) {
             const QRegularExpressionMatch idMatch = idRx.match(l);
-            QTC_ASSERT(idMatch.hasMatch(), return RevisionInfo());
+            QTC_ASSERT(idMatch.hasMatch(), return {});
             revisionId = idMatch.captured(1);
 
         } else if (l.startsWith("parent: ", Qt::CaseInsensitive)){
@@ -427,12 +426,12 @@ RevisionInfo FossilClient::synchronousRevisionQuery(const FilePath &workingDirec
     }
 
     // make sure id at least partially matches the retrieved revisionId
-    QTC_ASSERT(revisionId.startsWith(id, Qt::CaseInsensitive), return RevisionInfo());
+    QTC_ASSERT(revisionId.startsWith(id, Qt::CaseInsensitive), return {});
 
     if (parentId.isEmpty())
         parentId = revisionId;  // root
 
-    return RevisionInfo(revisionId, parentId, mergeParentIds, commentMsg, committer);
+    return {revisionId, parentId, mergeParentIds, commentMsg, committer};
 }
 
 QStringList FossilClient::synchronousTagQuery(const FilePath &workingDirectory, const QString &id)
@@ -442,41 +441,33 @@ QStringList FossilClient::synchronousTagQuery(const FilePath &workingDirectory,
     // Tag list includes branch names.
 
     if (workingDirectory.isEmpty())
-        return QStringList();
+        return {};
 
     QStringList args({"tag", "list"});
-
     if (!id.isEmpty())
         args << id;
-
     const CommandResult result = vcsSynchronousExec(workingDirectory, args);
     if (result.result() != ProcessResult::FinishedWithSuccess)
-        return QStringList();
+        return {};
 
-    const QString output = sanitizeFossilOutput(result.cleanedStdOut());
-
-    return output.split('\n', Qt::SkipEmptyParts);
+    return sanitizeFossilOutput(result.cleanedStdOut()).split('\n', Qt::SkipEmptyParts);
 }
 
 RepositorySettings FossilClient::synchronousSettingsQuery(const FilePath &workingDirectory)
 {
     if (workingDirectory.isEmpty())
-        return RepositorySettings();
+        return {};
 
-    RepositorySettings repoSettings;
+    const CommandResult result = vcsSynchronousExec(workingDirectory, QStringList{"settings"});
+    if (result.result() != ProcessResult::FinishedWithSuccess)
+        return {};
+    const QString output = sanitizeFossilOutput(result.cleanedStdOut());
 
+    RepositorySettings repoSettings;
     repoSettings.user = synchronousUserDefaultQuery(workingDirectory);
     if (repoSettings.user.isEmpty())
         repoSettings.user = settings().userName.value();
 
-    const QStringList args("settings");
-
-    const CommandResult result = vcsSynchronousExec(workingDirectory, args);
-    if (result.result() != ProcessResult::FinishedWithSuccess)
-        return RepositorySettings();
-
-    const QString output = sanitizeFossilOutput(result.cleanedStdOut());
-
     for (const QString &line : output.split('\n', Qt::SkipEmptyParts)) {
         // parse settings line:
         // <property> <(local|global)> <value>
@@ -507,8 +498,8 @@ RepositorySettings FossilClient::synchronousSettingsQuery(const FilePath &workin
     return repoSettings;
 }
 
-bool FossilClient::synchronousSetSetting(const FilePath &workingDirectory,
-                                         const QString &property, const QString &value, bool isGlobal)
+bool FossilClient::synchronousSetSetting(const FilePath &workingDirectory, const QString &property,
+                                         const QString &value, bool isGlobal)
 {
     // set a repository property to the given value
     // if no value is given, unset the property
@@ -529,7 +520,6 @@ bool FossilClient::synchronousSetSetting(const FilePath &workingDirectory,
             == ProcessResult::FinishedWithSuccess;
 }
 
-
 bool FossilClient::synchronousConfigureRepository(const FilePath &workingDirectory, const RepositorySettings &newSettings,
                                                   const RepositorySettings &currentSettings)
 {
@@ -540,20 +530,17 @@ bool FossilClient::synchronousConfigureRepository(const FilePath &workingDirecto
     const bool applyAll = (currentSettings == RepositorySettings());
 
     if (!newSettings.user.isEmpty()
-        && (applyAll
-            || newSettings.user != currentSettings.user)
-        && !synchronousSetUserDefault(workingDirectory, newSettings.user)){
+        && (applyAll || newSettings.user != currentSettings.user)
+        && !synchronousSetUserDefault(workingDirectory, newSettings.user)) {
         return false;
     }
 
-    if ((applyAll
-         || newSettings.sslIdentityFile != currentSettings.sslIdentityFile)
-        && !synchronousSetSetting(workingDirectory, "ssl-identity", newSettings.sslIdentityFile)){
+    if ((applyAll || newSettings.sslIdentityFile != currentSettings.sslIdentityFile)
+        && !synchronousSetSetting(workingDirectory, "ssl-identity", newSettings.sslIdentityFile)) {
         return false;
     }
 
-    if (applyAll
-        || newSettings.autosync != currentSettings.autosync) {
+    if (applyAll || newSettings.autosync != currentSettings.autosync) {
         QString value;
         switch (newSettings.autosync) {
         case RepositorySettings::AutosyncOff:
@@ -577,17 +564,13 @@ bool FossilClient::synchronousConfigureRepository(const FilePath &workingDirecto
 QString FossilClient::synchronousUserDefaultQuery(const FilePath &workingDirectory)
 {
     if (workingDirectory.isEmpty())
-        return QString();
-
-    const QStringList args({"user", "default"});
+        return {};
 
-    const CommandResult result = vcsSynchronousExec(workingDirectory, args);
+    const CommandResult result = vcsSynchronousExec(workingDirectory, {"user", "default"});
     if (result.result() != ProcessResult::FinishedWithSuccess)
-        return QString();
-
-    QString output = sanitizeFossilOutput(result.cleanedStdOut());
+        return {};
 
-    return output.trimmed();
+    return sanitizeFossilOutput(result.cleanedStdOut()).trimmed();
 }
 
 bool FossilClient::synchronousSetUserDefault(const FilePath &workingDirectory, const QString &userName)
@@ -604,19 +587,16 @@ bool FossilClient::synchronousSetUserDefault(const FilePath &workingDirectory, c
 QString FossilClient::synchronousGetRepositoryURL(const FilePath &workingDirectory)
 {
     if (workingDirectory.isEmpty())
-        return QString();
-
-    const QStringList args("remote-url");
+        return {};
 
-    const CommandResult result = vcsSynchronousExec(workingDirectory, args);
+    const CommandResult result = vcsSynchronousExec(workingDirectory, QStringList{"remote-url"});
     if (result.result() != ProcessResult::FinishedWithSuccess)
-        return QString();
-
+        return {};
     const QString output = sanitizeFossilOutput(result.cleanedStdOut()).trimmed();
 
     // Fossil returns "off" when no remote-url is set.
-    if (output.isEmpty() || output.toLower() == "off")
-        return QString();
+    if (output.toLower() == "off")
+        return {};
 
     return output;
 }
@@ -624,15 +604,10 @@ QString FossilClient::synchronousGetRepositoryURL(const FilePath &workingDirecto
 QString FossilClient::synchronousTopic(const FilePath &workingDirectory)
 {
     if (workingDirectory.isEmpty())
-        return QString();
+        return {};
 
     // return current branch name
-
-    const BranchInfo branchInfo = synchronousCurrentBranch(workingDirectory);
-    if (branchInfo.name().isEmpty())
-        return QString();
-
-    return branchInfo.name();
+    return synchronousCurrentBranch(workingDirectory).name;
 }
 
 bool FossilClient::synchronousCreateRepository(const FilePath &workingDirectory, const QStringList &extraOptions)
@@ -663,40 +638,24 @@ bool FossilClient::synchronousCreateRepository(const FilePath &workingDirectory,
     CommandResult result = vcsSynchronousExec(workingDirectory, args);
     if (result.result() != ProcessResult::FinishedWithSuccess)
         return false;
-
-    QString output = sanitizeFossilOutput(result.cleanedStdOut());
-    outputWindow->append(output);
+    outputWindow->append(sanitizeFossilOutput(result.cleanedStdOut()));
 
     // check out the created repository file into the working directory
-
-    args.clear();
-    output.clear();
-
-    args << "open" << repoFilePath.toUserOutput();
-    result = vcsSynchronousExec(workingDirectory, args);
+    result = vcsSynchronousExec(workingDirectory, {"open", repoFilePath.toUserOutput()});
     if (result.result() != ProcessResult::FinishedWithSuccess)
         return false;
-
-    output = sanitizeFossilOutput(result.cleanedStdOut());
-    outputWindow->append(output);
+    outputWindow->append(sanitizeFossilOutput(result.cleanedStdOut()));
 
     // set user default to admin if specified
-
     if (!adminUser.isEmpty()) {
-        args.clear();
-        output.clear();
-
-        args << "user" << "default" << adminUser << "--user" << adminUser;
-        result = vcsSynchronousExec(workingDirectory, args);
+        result = vcsSynchronousExec(workingDirectory,
+                                    {"user", "default", adminUser, "--user", adminUser});
         if (result.result() != ProcessResult::FinishedWithSuccess)
             return false;
-
-        QString output = sanitizeFossilOutput(result.cleanedStdOut());
-        outputWindow->append(output);
+        outputWindow->append(sanitizeFossilOutput(result.cleanedStdOut()));
     }
 
     resetCachedVcsInfo(workingDirectory);
-
     return true;
 }
 
@@ -794,8 +753,6 @@ VcsBaseEditorWidget *FossilClient::annotate(const FilePath &workingDir, const QS
     if (VcsBaseEditorConfig *editorConfig = fossilEditor->editorConfig())
         effectiveArgs = editorConfig->arguments();
 
-    VcsCommand *cmd = createCommand(workingDir, fossilEditor);
-
     // here we introduce a "|BLAME|" meta-option to allow both annotate and blame modes
     int pos = effectiveArgs.indexOf("|BLAME|");
     if (pos != -1) {
@@ -803,10 +760,8 @@ VcsBaseEditorWidget *FossilClient::annotate(const FilePath &workingDir, const QS
         effectiveArgs.removeAt(pos);
     }
     QStringList args(vcsCmdString);
-    if (!revision.isEmpty()
-        && supportedFeatures().testFlag(AnnotateRevisionFeature))
+    if (!revision.isEmpty() && supportedFeatures().testFlag(AnnotateRevisionFeature))
         args << "-r" << revision;
-
     args << effectiveArgs << file;
 
     // When version list requested, ignore the source line.
@@ -814,7 +769,7 @@ VcsBaseEditorWidget *FossilClient::annotate(const FilePath &workingDir, const QS
         lineNumber = -1;
     editor->setDefaultLineNumber(lineNumber);
 
-    enqueueJob(cmd, args);
+    enqueueJob(createCommand(workingDir, fossilEditor), args);
     return fossilEditor;
 }
 
@@ -833,8 +788,7 @@ FilePath FossilClient::findTopLevelForFile(const FilePath &file) const
 
 bool FossilClient::managesFile(const FilePath &workingDirectory, const QString &fileName) const
 {
-    const QStringList args({"finfo", fileName});
-    const CommandResult result = vcsSynchronousExec(workingDirectory, args);
+    const CommandResult result = vcsSynchronousExec(workingDirectory, {"finfo", fileName});
     if (result.result() != ProcessResult::FinishedWithSuccess)
         return false;
     QString output = sanitizeFossilOutput(result.cleanedStdOut());
@@ -853,8 +807,7 @@ unsigned int FossilClient::binaryVersion() const
 
     // Invalidate cache on failed version result.
     // Assume that fossil client options have been changed and will change again.
-    if (!cachedBinaryVersion
-        || currentBinaryPath != cachedBinaryPath) {
+    if (!cachedBinaryVersion || currentBinaryPath != cachedBinaryPath) {
         cachedBinaryVersion = synchronousBinaryVersion();
         if (cachedBinaryVersion)
             cachedBinaryPath = currentBinaryPath;
@@ -903,18 +856,11 @@ FossilClient::SupportedFeatures FossilClient::supportedFeatures() const
 
 void FossilClient::view(const QString &source, const QString &id, const QStringList &extraOptions)
 {
-    QStringList args("diff");
-
     const FilePath fPath = FilePath::fromString(source);
     const FilePath workingDirectory = fPath.isFile() ? fPath.absolutePath() : fPath;
 
-    RevisionInfo revisionInfo = synchronousRevisionQuery(workingDirectory,id);
-
-    args << "--from" << revisionInfo.parentId
-         << "--to" << revisionInfo.id
-         << "-v"
-         << extraOptions;
-
+    const RevisionInfo revisionInfo = synchronousRevisionQuery(workingDirectory, id);
+    const QStringList args{"diff", "--from", revisionInfo.parentId, "--to", revisionInfo.id, "-v"};
     const Id kind = vcsEditorKind(DiffCommand);
     const QString title = vcsEditorTitle(vcsCommandString(DiffCommand), id);
 
@@ -922,7 +868,7 @@ void FossilClient::view(const QString &source, const QString &id, const QStringL
                                                   VcsBaseEditor::getCodec(source), "view", id);
     editor->setWorkingDirectory(workingDirectory);
 
-    enqueueJob(createCommand(workingDirectory, editor), args);
+    enqueueJob(createCommand(workingDirectory, editor), args + extraOptions);
 }
 
 class FossilLogHighlighter : QSyntaxHighlighter
@@ -1081,12 +1027,11 @@ void FossilClient::revertFile(const FilePath &workingDir,
     QStringList args(vcsCommandString(RevertCommand));
     if (!revision.isEmpty())
         args << "-r" << revision;
-
     args << extraOptions << file;
 
     // Indicate file list
     VcsCommand *cmd = createCommand(workingDir);
-    const QStringList files = QStringList(workingDir.toString() + "/" + file);
+    const QStringList files = {workingDir.toString() + "/" + file};
     connect(cmd, &VcsCommand::done, this, [this, files, cmd] {
         if (cmd->result() == ProcessResult::FinishedWithSuccess)
             emit changed(files);
@@ -1103,15 +1048,10 @@ void FossilClient::revertAll(const FilePath &workingDir, const QString &revision
     //       Thus undo for whole tree revert should not be possible.
 
     QStringList args;
-    if (revision.isEmpty()) {
-        args << vcsCommandString(RevertCommand)
-             << extraOptions;
-
-    } else {
-        args << "checkout" << revision
-             << "--force"
-             << extraOptions;
-    }
+    if (revision.isEmpty())
+        args << vcsCommandString(RevertCommand) << extraOptions;
+    else
+        args << "checkout" << revision << "--force" << extraOptions;
 
     // Indicate repository change
     VcsCommand *cmd = createCommand(workingDir);
@@ -1146,10 +1086,9 @@ QString FossilClient::vcsCommandString(VcsCommandTag cmd) const
     // otherwise return baseclient command
 
     switch (cmd) {
-    case RemoveCommand: return QString("rm");
-    case MoveCommand: return QString("mv");
-    case LogCommand: return QString("timeline");
-
+    case RemoveCommand: return "rm";
+    case MoveCommand: return "mv";
+    case LogCommand: return "timeline";
     default: return VcsBaseClient::vcsCommandString(cmd);
     }
 }
@@ -1164,7 +1103,7 @@ Id FossilClient::vcsEditorKind(VcsCommandTag cmd) const
     case LogCommand:
         return Constants::FILELOG_ID;
     default:
-        return Id();
+        return {};
     }
 }
 
@@ -1192,7 +1131,7 @@ FossilClient::StatusItem FossilClient::parseStatusLine(const QString &line) cons
     int pos = line.indexOf(' ');
 
     if (line.isEmpty() || pos < 1)
-        return StatusItem();
+        return {};
 
     QString label(line.left(pos));
     QString flags;
@@ -1228,16 +1167,15 @@ FossilClient::StatusItem FossilClient::parseStatusLine(const QString &line) cons
     else if (label == "NOT_A_FILE")
         flags = Constants::FSTATUS_UNKNOWN;
 
-
     if (flags.isEmpty())
-        return StatusItem();
+        return {};
 
     // adjust the position to the last space before the file name
-    for (int size = line.size(); (pos+1) < size && line[pos+1].isSpace(); ++pos) {}
+    for (int size = line.size(); (pos + 1) < size && line[pos + 1].isSpace(); ++pos)
+        ;
 
     item.flags = flags;
     item.file = line.mid(pos + 1);
-
     return item;
 }
 
diff --git a/plugins/fossil/fossilclient.h b/plugins/fossil/fossilclient.h
index 6af10c0..bc5d0bd 100644
--- a/plugins/fossil/fossilclient.h
+++ b/plugins/fossil/fossilclient.h
@@ -70,56 +70,51 @@ public:
     BranchInfo synchronousCurrentBranch(const Utils::FilePath &workingDirectory);
     QList<BranchInfo> synchronousBranchQuery(const Utils::FilePath &workingDirectory);
     RevisionInfo synchronousRevisionQuery(const Utils::FilePath &workingDirectory,
-                                          const QString &id = QString(),
-                                          bool getCommentMsg = false) const;
+                                          const QString &id = {}, bool getCommentMsg = false) const;
     QStringList synchronousTagQuery(const Utils::FilePath &workingDirectory, const QString &id = {});
     RepositorySettings synchronousSettingsQuery(const Utils::FilePath &workingDirectory);
     bool synchronousSetSetting(const Utils::FilePath &workingDirectory, const QString &property,
-                               const QString &value = QString(), bool isGlobal = false);
-    bool synchronousConfigureRepository(const Utils::FilePath &workingDirectory, const RepositorySettings &newSettings,
-                                        const RepositorySettings &currentSettings = RepositorySettings());
+                               const QString &value = {}, bool isGlobal = false);
+    bool synchronousConfigureRepository(const Utils::FilePath &workingDirectory,
+                                        const RepositorySettings &newSettings,
+                                        const RepositorySettings &currentSettings = {});
     QString synchronousUserDefaultQuery(const Utils::FilePath &workingDirectory);
     bool synchronousSetUserDefault(const Utils::FilePath &workingDirectory, const QString &userName);
     QString synchronousGetRepositoryURL(const Utils::FilePath &workingDirectory);
     QString synchronousTopic(const Utils::FilePath &workingDirectory);
     bool synchronousCreateRepository(const Utils::FilePath &workingDirectory,
-                                     const QStringList &extraOptions = QStringList()) final;
-    bool synchronousMove(const Utils::FilePath &workingDir,
-                         const QString &from, const QString &to,
-                         const QStringList &extraOptions = QStringList()) final;
-    bool synchronousPull(const Utils::FilePath &workingDir,
-                         const QString &srcLocation,
-                         const QStringList &extraOptions = QStringList()) final;
-    bool synchronousPush(const Utils::FilePath &workingDir,
-                         const QString &dstLocation,
-                         const QStringList &extraOptions = QStringList()) final;
+                                     const QStringList &extraOptions = {}) final;
+    bool synchronousMove(const Utils::FilePath &workingDir, const QString &from, const QString &to,
+                         const QStringList &extraOptions = {}) final;
+    bool synchronousPull(const Utils::FilePath &workingDir, const QString &srcLocation,
+                         const QStringList &extraOptions = {}) final;
+    bool synchronousPush(const Utils::FilePath &workingDir, const QString &dstLocation,
+                         const QStringList &extraOptions = {}) final;
     void commit(const Utils::FilePath &repositoryRoot, const QStringList &files,
-                const QString &commitMessageFile, const QStringList &extraOptions = QStringList()) final;
-    VcsBase::VcsBaseEditorWidget *annotate(
-            const Utils::FilePath &workingDir, const QString &file, const QString &revision = {},
-            int lineNumber = -1, const QStringList &extraOptions = QStringList()) final;
-    void log(const Utils::FilePath &workingDir, const QStringList &files = QStringList(),
-             const QStringList &extraOptions = QStringList(),
-             bool enableAnnotationContextMenu = false) final;
-    void logCurrentFile(const Utils::FilePath &workingDir, const QStringList &files = QStringList(),
-                        const QStringList &extraOptions = QStringList(),
+                const QString &commitMessageFile, const QStringList &extraOptions = {}) final;
+    VcsBase::VcsBaseEditorWidget *annotate(const Utils::FilePath &workingDir, const QString &file,
+                                           const QString &revision = {}, int lineNumber = -1,
+                                           const QStringList &extraOptions = {}) final;
+    void log(const Utils::FilePath &workingDir, const QStringList &files = {},
+             const QStringList &extraOptions = {}, bool enableAnnotationContextMenu = false) final;
+    void logCurrentFile(const Utils::FilePath &workingDir, const QStringList &files = {},
+                        const QStringList &extraOptions = {},
                         bool enableAnnotationContextMenu = false);
     void revertFile(const Utils::FilePath &workingDir, const QString &file,
-                    const QString &revision = QString(),
-                    const QStringList &extraOptions = QStringList()) final;
-    void revertAll(const Utils::FilePath &workingDir, const QString &revision = QString(),
-                   const QStringList &extraOptions = QStringList()) final;
+                    const QString &revision = {}, const QStringList &extraOptions = {}) final;
+    void revertAll(const Utils::FilePath &workingDir, const QString &revision = {},
+                   const QStringList &extraOptions = {}) final;
     bool isVcsFileOrDirectory(const Utils::FilePath &filePath) const;
     Utils::FilePath findTopLevelForFile(const Utils::FilePath &file) const final;
     bool managesFile(const Utils::FilePath &workingDirectory, const QString &fileName) const;
     unsigned int binaryVersion() const;
     QString binaryVersionString() const;
     SupportedFeatures supportedFeatures() const;
-    void view(const QString &source, const QString &id,
-              const QStringList &extraOptions = QStringList()) final;
+    void view(const QString &source, const QString &id, const QStringList &extraOptions = {}) final;
 
 private:
-    static QList<BranchInfo> branchListFromOutput(const QString &output, const BranchInfo::BranchFlags defaultFlags = {});
+    static QList<BranchInfo> branchListFromOutput(const QString &output,
+                                                  const BranchInfo::BranchFlags defaultFlags = {});
     static QStringList parseRevisionCommentLine(const QString &commentLine);
 
     QString sanitizeFossilOutput(const QString &output) const;
diff --git a/plugins/fossil/fossilcommitwidget.cpp b/plugins/fossil/fossilcommitwidget.cpp
index d53ac59..beef90f 100644
--- a/plugins/fossil/fossilcommitwidget.cpp
+++ b/plugins/fossil/fossilcommitwidget.cpp
@@ -33,11 +33,9 @@
 #include <utils/completingtextedit.h>
 #include <utils/qtcassert.h>
 
-#include <QSyntaxHighlighter>
-#include <QTextEdit>
-
-#include <QRegularExpression>
 #include <QDir>
+#include <QRegularExpression>
+#include <QSyntaxHighlighter>
 
 namespace Fossil {
 namespace Internal {
@@ -101,7 +99,7 @@ void FossilCommitWidget::setFields(const QString &repoPath, const BranchInfo &br
                                    const QStringList &tags, const QString &userName)
 {
     m_commitPanelUi.localRootLineEdit->setText(QDir::toNativeSeparators(repoPath));
-    m_commitPanelUi.currentBranchLineEdit->setText(branch.name());
+    m_commitPanelUi.currentBranchLineEdit->setText(branch.name);
     const QString tagsText = tags.join(", ");
     m_commitPanelUi.currentTagsLineEdit->setText(tagsText);
     m_commitPanelUi.authorLineEdit->setText(userName);
@@ -111,29 +109,21 @@ void FossilCommitWidget::setFields(const QString &repoPath, const BranchInfo &br
 
 QString FossilCommitWidget::newBranch() const
 {
-    const QString branchName = m_commitPanelUi.branchLineEdit->text().trimmed();
-    return branchName;
+    return m_commitPanelUi.branchLineEdit->text().trimmed();
 }
 
 QStringList FossilCommitWidget::tags() const
 {
     QString tagsText = m_commitPanelUi.tagsLineEdit->text().trimmed();
     if (tagsText.isEmpty())
-        return QStringList();
+        return {};
 
-    tagsText.replace(',', ' ');
-    const QStringList tags = tagsText.split(' ', Qt::SkipEmptyParts);
-    return tags;
+    return tagsText.replace(',', ' ').split(' ', Qt::SkipEmptyParts);
 }
 
 QString FossilCommitWidget::committer() const
 {
-    const QString author = m_commitPanelUi.authorLineEdit->text();
-    if (author.isEmpty())
-        return QString();
-
-    const QString user = author;
-    return user;
+    return m_commitPanelUi.authorLineEdit->text();
 }
 
 bool FossilCommitWidget::isPrivateOptionEnabled() const
diff --git a/plugins/fossil/fossilcommitwidget.h b/plugins/fossil/fossilcommitwidget.h
index 689eb45..aa84e4e 100644
--- a/plugins/fossil/fossilcommitwidget.h
+++ b/plugins/fossil/fossilcommitwidget.h
@@ -50,8 +50,8 @@ class FossilCommitWidget : public VcsBase::SubmitEditorWidget
 public:
     FossilCommitWidget();
 
-    void setFields(const QString &repoPath,
-                   const BranchInfo &newBranch, const QStringList &tags, const QString &userName);
+    void setFields(const QString &repoPath, const BranchInfo &newBranch, const QStringList &tags,
+                   const QString &userName);
 
     QString newBranch() const;
     QStringList tags() const;
diff --git a/plugins/fossil/fossileditor.cpp b/plugins/fossil/fossileditor.cpp
index adeacda..92145b7 100644
--- a/plugins/fossil/fossileditor.cpp
+++ b/plugins/fossil/fossileditor.cpp
@@ -29,17 +29,10 @@
 #include "fossilplugin.h"
 #include "fossilclient.h"
 
-#include <coreplugin/editormanager/editormanager.h>
 #include <utils/qtcassert.h>
-#include <utils/qtcprocess.h>
-#include <vcsbase/diffandloghighlighter.h>
 
 #include <QRegularExpression>
-#include <QString>
 #include <QTextCursor>
-#include <QTextBlock>
-#include <QDir>
-#include <QFileInfo>
 
 namespace Fossil {
 namespace Internal {
@@ -78,11 +71,11 @@ QString FossilEditorWidget::changeUnderCursor(const QTextCursor &cursorIn) const
     cursor.select(QTextCursor::WordUnderCursor);
     if (cursor.hasSelection()) {
         const QString change = cursor.selectedText();
-        QRegularExpressionMatch exactChangesetIdMatch = d->m_exactChangesetId.match(change);
+        const QRegularExpressionMatch exactChangesetIdMatch = d->m_exactChangesetId.match(change);
         if (exactChangesetIdMatch.hasMatch())
             return change;
     }
-    return QString();
+    return {};
 }
 
 QString FossilEditorWidget::decorateVersion(const QString &revision) const
@@ -92,9 +85,8 @@ QString FossilEditorWidget::decorateVersion(const QString &revision) const
 
     const Utils::FilePath workingDirectory = Utils::FilePath::fromString(source()).parentDir();
     const FossilClient *client = FossilPlugin::client();
-    RevisionInfo revisionInfo =
-            client->synchronousRevisionQuery(workingDirectory, revision, true);
-
+    const RevisionInfo revisionInfo = client->synchronousRevisionQuery(workingDirectory, revision,
+                                                                       true);
     // format: 'revision (committer "comment...")'
     QString output = revision.left(shortChangesetIdSize)
             + " (" + revisionInfo.committer
@@ -110,15 +102,13 @@ QString FossilEditorWidget::decorateVersion(const QString &revision) const
 
 QStringList FossilEditorWidget::annotationPreviousVersions(const QString &revision) const
 {
-    QStringList revisions;
     const Utils::FilePath workingDirectory = Utils::FilePath::fromString(source()).parentDir();
     const FossilClient *client = FossilPlugin::client();
-    RevisionInfo revisionInfo =
-            client->synchronousRevisionQuery(workingDirectory, revision);
+    const RevisionInfo revisionInfo = client->synchronousRevisionQuery(workingDirectory, revision);
     if (revisionInfo.parentId.isEmpty())
-        return QStringList();
+        return {};
 
-    revisions.append(revisionInfo.parentId);
+    QStringList revisions{revisionInfo.parentId};
     revisions.append(revisionInfo.mergeParentIds);
     return revisions;
 }
diff --git a/plugins/fossil/fossilplugin.cpp b/plugins/fossil/fossilplugin.cpp
index c4281e4..643a213 100644
--- a/plugins/fossil/fossilplugin.cpp
+++ b/plugins/fossil/fossilplugin.cpp
@@ -298,11 +298,6 @@ FossilClient *FossilPlugin::client()
     return &dd->m_client;
 }
 
-void FossilPlugin::showCommitWidget(const QList<VcsBaseClient::StatusItem> &status)
-{
-    dd->showCommitWidget(status);
-}
-
 FossilPluginPrivate::FossilPluginPrivate()
     : VcsBase::VcsBasePluginPrivate(Core::Context(Constants::FOSSIL_CONTEXT))
 {
@@ -420,7 +415,7 @@ void FossilPluginPrivate::annotateCurrentFile()
     const VcsBase::VcsBasePluginState state = currentState();
     QTC_ASSERT(state.hasFile(), return);
     const int lineNumber = VcsBase::VcsBaseEditor::lineNumberOfCurrentEditor(state.currentFile());
-    m_client.annotate(state.currentFileTopLevel(), state.relativeCurrentFile(), QString(), lineNumber);
+    m_client.annotate(state.currentFileTopLevel(), state.relativeCurrentFile(), {}, lineNumber);
 }
 
 void FossilPluginPrivate::diffCurrentFile()
@@ -524,7 +519,7 @@ void FossilPluginPrivate::logRepository()
     if (features.testFlag(FossilClient::TimelineWidthFeature))
         extraOptions << "-W" << QString::number(m_client.settings().timelineWidth.value());
 
-    m_client.log(state.topLevel(), QStringList(), extraOptions);
+    m_client.log(state.topLevel(), {}, extraOptions);
 }
 
 void FossilPluginPrivate::revertAll()
@@ -688,12 +683,8 @@ void FossilPluginPrivate::commit()
     QTC_ASSERT(state.hasTopLevel(), return);
 
     m_submitRepository = state.topLevel();
-
-    connect(&m_client, &VcsBaseClient::parsedStatus,
-            this, &FossilPluginPrivate::showCommitWidget);
-
-    QStringList extraOptions;
-    m_client.emitParsedStatus(m_submitRepository, extraOptions);
+    connect(&m_client, &VcsBaseClient::parsedStatus, this, &FossilPluginPrivate::showCommitWidget);
+    m_client.emitParsedStatus(m_submitRepository, {});
 }
 
 void FossilPluginPrivate::showCommitWidget(const QList<VcsBase::VcsBaseClient::StatusItem> &status)
@@ -738,7 +729,7 @@ void FossilPluginPrivate::showCommitWidget(const QList<VcsBase::VcsBaseClient::S
     const QString currentUser = m_client.synchronousUserDefaultQuery(m_submitRepository);
     QStringList tags = m_client.synchronousTagQuery(m_submitRepository, currentRevision.id);
     // Fossil includes branch name in tag list -- remove.
-    tags.removeAll(currentBranch.name());
+    tags.removeAll(currentBranch.name);
     commitEditor->setFields(m_submitRepository.toString(), currentBranch, tags, currentUser, status);
 
     connect(commitEditor, &VcsBase::VcsBaseSubmitEditor::diffSelectedFiles,
@@ -853,7 +844,8 @@ bool FossilPluginPrivate::submitEditorAboutToClose()
             extraOptions << "--branch" << enquotedBranch;
         }
         // Tags
-        foreach (QString tag, commitWidget->tags()) {
+        const QStringList tags = commitWidget->tags();
+        for (const QString &tag : tags) {
             extraOptions << "--tag" << tag;
         }
 
@@ -886,7 +878,7 @@ void FossilPluginPrivate::updateActions(VcsBase::VcsBasePluginPrivate::ActionSta
     m_revertFile->setParameter(filename);
     m_statusFile->setParameter(filename);
 
-    foreach (QAction *repoAction, m_repositoryActionList)
+    for (QAction *repoAction : qAsConst(m_repositoryActionList))
         repoAction->setEnabled(repoEnabled);
 }
 
@@ -979,8 +971,7 @@ bool FossilPluginPrivate::vcsMove(const FilePath &from, const FilePath &to)
 {
     const QFileInfo fromInfo = from.toFileInfo();
     const QFileInfo toInfo = to.toFileInfo();
-    return m_client.synchronousMove(from.absolutePath(),
-                                    fromInfo.absoluteFilePath(),
+    return m_client.synchronousMove(from.absolutePath(), fromInfo.absoluteFilePath(),
                                     toInfo.absoluteFilePath());
 }
 
@@ -991,18 +982,20 @@ bool FossilPluginPrivate::vcsCreateRepository(const FilePath &directory)
 
 void FossilPluginPrivate::vcsAnnotate(const FilePath &filePath, int line)
 {
-    m_client.annotate(filePath.absolutePath(), filePath.fileName(), QString(), line);
+    m_client.annotate(filePath.absolutePath(), filePath.fileName(), {}, line);
 }
 
-void FossilPluginPrivate::vcsDescribe(const FilePath &source, const QString &id) { m_client.view(source.toString(), id); }
+void FossilPluginPrivate::vcsDescribe(const FilePath &source, const QString &id)
+{
+    m_client.view(source.toString(), id);
+}
 
 VcsCommand *FossilPluginPrivate::createInitialCheckoutCommand(const QString &sourceUrl,
                                                               const FilePath &baseDirectory,
                                                               const QString &localName,
                                                               const QStringList &extraArgs)
 {
-    QMap<QString, QString> options;
-    FossilJsExtension::parseArgOptions(extraArgs, options);
+    const QMap<QString, QString> options = FossilJsExtension::parseArgOptions(extraArgs);
 
     // Two operating modes:
     //  1) CloneCheckout:
diff --git a/plugins/fossil/fossilplugin.h b/plugins/fossil/fossilplugin.h
index 436f237..4046a83 100644
--- a/plugins/fossil/fossilplugin.h
+++ b/plugins/fossil/fossilplugin.h
@@ -50,8 +50,6 @@ public:
     static const FossilSettings &settings();
     static FossilClient *client();
 
-    static void showCommitWidget(const QList<VcsBase::VcsBaseClient::StatusItem> &status);
-
 #ifdef WITH_TESTS
 private slots:
     void testDiffFileResolving_data();
diff --git a/plugins/fossil/fossilsettings.cpp b/plugins/fossil/fossilsettings.cpp
index 27296ad..72bb99d 100644
--- a/plugins/fossil/fossilsettings.cpp
+++ b/plugins/fossil/fossilsettings.cpp
@@ -26,7 +26,6 @@
 #include "fossilsettings.h"
 
 #include "constants.h"
-#include "fossilclient.h"
 
 #include <coreplugin/icore.h>
 
@@ -119,11 +118,6 @@ FossilSettings::FossilSettings()
         "Choose 0 to see all entries."));
 };
 
-RepositorySettings::RepositorySettings()
-    : autosync(AutosyncOn)
-{
-}
-
 // OptionsPage
 
 class OptionsPageWidget final : public Core::IOptionsPageWidget
diff --git a/plugins/fossil/fossilsettings.h b/plugins/fossil/fossilsettings.h
index e686c8f..f06e683 100644
--- a/plugins/fossil/fossilsettings.h
+++ b/plugins/fossil/fossilsettings.h
@@ -51,20 +51,18 @@ public:
 
 struct RepositorySettings
 {
-    enum AutosyncMode {AutosyncOff = 0, AutosyncOn = 1, AutosyncPullOnly};
+    enum AutosyncMode {AutosyncOff, AutosyncOn, AutosyncPullOnly};
 
     QString user;
-    AutosyncMode autosync;
     QString sslIdentityFile;
-
-    RepositorySettings();
+    AutosyncMode autosync = AutosyncOn;
 };
 
-inline bool operator== (const RepositorySettings &lh, const RepositorySettings &rh)
+inline bool operator==(const RepositorySettings &lh, const RepositorySettings &rh)
 {
-    return (lh.user == rh.user
-            && lh.autosync == rh.autosync
-            && lh.sslIdentityFile == rh.sslIdentityFile);
+    return (lh.user == rh.user &&
+            lh.sslIdentityFile == rh.sslIdentityFile &&
+            lh.autosync == rh.autosync);
 }
 
 class OptionsPage : public Core::IOptionsPage
diff --git a/plugins/fossil/revisioninfo.cpp b/plugins/fossil/revisioninfo.cpp
deleted file mode 100644
index aeacfdc..0000000
--- a/plugins/fossil/revisioninfo.cpp
+++ /dev/null
@@ -1,42 +0,0 @@
-/****************************************************************************
-**
-** Copyright (c) 2018 Artur Shepilko
-** Contact: https://www.qt.io/licensing/
-**
-** This file is part of Qt Creator.
-**
-** Commercial License Usage
-** Licensees holding valid commercial Qt licenses may use this file in
-** accordance with the commercial license agreement provided with the
-** Software or, alternatively, in accordance with the terms contained in
-** a written agreement between you and The Qt Company. For licensing terms
-** and conditions see https://www.qt.io/terms-conditions. For further
-** information use the contact form at https://www.qt.io/contact-us.
-**
-** GNU General Public License Usage
-** Alternatively, this file may be used under the terms of the GNU
-** General Public License version 3 as published by the Free Software
-** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT
-** included in the packaging of this file. Please review the following
-** information to ensure the GNU General Public License requirements will
-** be met: https://www.gnu.org/licenses/gpl-3.0.html.
-**
-****************************************************************************/
-
-#include "revisioninfo.h"
-
-namespace Fossil {
-namespace Internal {
-
-RevisionInfo::RevisionInfo(const QString &revisionId, const QString &parent,
-                           const QStringList &mergeParents, const QString &comment,
-                           const QString &user) :
-    id(revisionId),
-    parentId(parent),
-    mergeParentIds(mergeParents),
-    commentMsg(comment),
-    committer(user)
-{ }
-
-} // namespace Internal
-} // namespace Fossil
diff --git a/plugins/fossil/revisioninfo.h b/plugins/fossil/revisioninfo.h
index 11cd4a7..984c61b 100644
--- a/plugins/fossil/revisioninfo.h
+++ b/plugins/fossil/revisioninfo.h
@@ -25,8 +25,6 @@
 
 #pragma once
 
-#include "branchinfo.h"
-
 #include <QString>
 #include <QStringList>
 
@@ -36,16 +34,11 @@ namespace Internal {
 class RevisionInfo
 {
 public:
-    explicit RevisionInfo(const QString &revisionId = QString(), const QString &parent = QString(),
-                          const QStringList &mergeParents = QStringList(),
-                          const QString &comment = QString(), const QString &user = QString());
-
     const QString id;
     const QString parentId;
     const QStringList mergeParentIds;
     const QString commentMsg;
     const QString committer;
-
 };
 
 } // namespace Internal
diff --git a/plugins/fossil/wizard/fossiljsextension.cpp b/plugins/fossil/wizard/fossiljsextension.cpp
index 701c55d..dad175d 100644
--- a/plugins/fossil/wizard/fossiljsextension.cpp
+++ b/plugins/fossil/wizard/fossiljsextension.cpp
@@ -52,16 +52,16 @@ public:
 };
 
 
-void FossilJsExtension::parseArgOptions(const QStringList &args, QMap<QString, QString> &options)
+QMap<QString, QString> FossilJsExtension::parseArgOptions(const QStringList &args)
 {
-    options.clear();
-
-    foreach (const QString &arg, args) {
-        if (arg.isEmpty()) continue;
-
-        QStringList opt = arg.split('|', Qt::KeepEmptyParts);
+    QMap<QString, QString> options;
+    for (const QString &arg : args) {
+        if (arg.isEmpty())
+            continue;
+        const QStringList opt = arg.split('|', Qt::KeepEmptyParts);
         options.insert(opt[0], opt.size() > 1 ? opt[1] : QString());
     }
+    return options;
 }
 
 FossilJsExtension::FossilJsExtension(FossilSettings *settings) :
diff --git a/plugins/fossil/wizard/fossiljsextension.h b/plugins/fossil/wizard/fossiljsextension.h
index 9e9ff02..3d04f23 100644
--- a/plugins/fossil/wizard/fossiljsextension.h
+++ b/plugins/fossil/wizard/fossiljsextension.h
@@ -42,7 +42,7 @@ class FossilJsExtension : public QObject
     Q_OBJECT
 
 public:
-    static void parseArgOptions(const QStringList &args, QMap<QString, QString> &options);
+    static QMap<QString, QString> parseArgOptions(const QStringList &args);
 
     FossilJsExtension(FossilSettings *settings);
     ~FossilJsExtension();