Git: Modernize

* Use auto
* Use override
* Use some member initializers

Change-Id: I3ca000d1c8e4d02331d58b85e68e4d771c636b29
Reviewed-by: André Hartmann <aha_1980@gmx.de>

37 files changed, 68 insertions, 74 deletions

diff --git a/src/plugins/git/annotationhighlighter.h b/src/plugins/git/annotationhighlighter.h
index 7f28ad26da..629b8c3f5e 100644
--- a/src/plugins/git/annotationhighlighter.h
+++ b/src/plugins/git/annotationhighlighter.h
@@ -36,7 +36,7 @@ class GitAnnotationHighlighter : public VcsBase::BaseAnnotationHighlighter
     Q_OBJECT
 public:
     explicit GitAnnotationHighlighter(const ChangeNumbers &changeNumbers,
-                                      QTextDocument *document = 0);
+                                      QTextDocument *document = nullptr);
 
 private:
     QString changeNumber(const QString &block) const override;
diff --git a/src/plugins/git/branchadddialog.cpp b/src/plugins/git/branchadddialog.cpp
index d3c122b974..2ea024bef4 100644
--- a/src/plugins/git/branchadddialog.cpp
+++ b/src/plugins/git/branchadddialog.cpp
@@ -46,7 +46,7 @@ namespace Internal {
 class BranchNameValidator : public QValidator
 {
 public:
-    BranchNameValidator(const QStringList &localBranches, QObject *parent = 0) :
+    BranchNameValidator(const QStringList &localBranches, QObject *parent = nullptr) :
         QValidator(parent),
         m_invalidChars(GitPlugin::invalidBranchAndRemoteNamePattern()),
         m_localBranches(localBranches)
diff --git a/src/plugins/git/branchcheckoutdialog.cpp b/src/plugins/git/branchcheckoutdialog.cpp
index c6a0db9a3c..97881c7839 100644
--- a/src/plugins/git/branchcheckoutdialog.cpp
+++ b/src/plugins/git/branchcheckoutdialog.cpp
@@ -33,9 +33,7 @@ BranchCheckoutDialog::BranchCheckoutDialog(QWidget *parent,
                                            const QString &currentBranch,
                                            const QString &nextBranch) :
     QDialog(parent),
-    m_ui(new Ui::BranchCheckoutDialog),
-    m_foundStashForNextBranch(false),
-    m_hasLocalChanges(true)
+    m_ui(new Ui::BranchCheckoutDialog)
 {
     m_ui->setupUi(this);
 
diff --git a/src/plugins/git/branchcheckoutdialog.h b/src/plugins/git/branchcheckoutdialog.h
index 755a132e84..882e5ded40 100644
--- a/src/plugins/git/branchcheckoutdialog.h
+++ b/src/plugins/git/branchcheckoutdialog.h
@@ -56,8 +56,8 @@ private:
     void updatePopStashCheckBox(bool moveChangesChecked);
 
     Ui::BranchCheckoutDialog *m_ui;
-    bool m_foundStashForNextBranch;
-    bool m_hasLocalChanges;
+    bool m_foundStashForNextBranch = false;
+    bool m_hasLocalChanges = true;
 };
 
 } // namespace Internal
diff --git a/src/plugins/git/changeselectiondialog.cpp b/src/plugins/git/changeselectiondialog.cpp
index 247427eed5..5a46b2cbbf 100644
--- a/src/plugins/git/changeselectiondialog.cpp
+++ b/src/plugins/git/changeselectiondialog.cpp
@@ -204,7 +204,7 @@ void ChangeSelectionDialog::terminateProcess()
     m_process->kill();
     m_process->waitForFinished();
     delete m_process;
-    m_process = 0;
+    m_process = nullptr;
 }
 
 void ChangeSelectionDialog::recalculateCompletion()
diff --git a/src/plugins/git/commitdata.cpp b/src/plugins/git/commitdata.cpp
index d8a2edc620..98876c678a 100644
--- a/src/plugins/git/commitdata.cpp
+++ b/src/plugins/git/commitdata.cpp
@@ -64,8 +64,6 @@ QString GitSubmitEditorPanelData::authorString() const
 
 CommitData::CommitData(CommitType type)
     : commitType(type)
-    , commitEncoding(0)
-    , enablePush(false)
 {
 }
 
diff --git a/src/plugins/git/commitdata.h b/src/plugins/git/commitdata.h
index 98e6fe9ca4..77a6f52a62 100644
--- a/src/plugins/git/commitdata.h
+++ b/src/plugins/git/commitdata.h
@@ -105,10 +105,10 @@ public:
 
     CommitType commitType;
     QString amendSHA1;
-    QTextCodec *commitEncoding;
+    QTextCodec *commitEncoding = nullptr;
     GitSubmitEditorPanelInfo panelInfo;
     GitSubmitEditorPanelData panelData;
-    bool enablePush;
+    bool enablePush = false;
 
     QList<StateFilePair> files;
 
diff --git a/src/plugins/git/gerrit/authenticationdialog.h b/src/plugins/git/gerrit/authenticationdialog.h
index 055c553b48..7556c5bd0b 100644
--- a/src/plugins/git/gerrit/authenticationdialog.h
+++ b/src/plugins/git/gerrit/authenticationdialog.h
@@ -45,7 +45,7 @@ class AuthenticationDialog : public QDialog
 
 public:
     AuthenticationDialog(GerritServer *server);
-    ~AuthenticationDialog();
+    ~AuthenticationDialog() override;
     bool isAuthenticated() const { return m_authenticated; }
 
 private:
diff --git a/src/plugins/git/gerrit/gerritdialog.cpp b/src/plugins/git/gerrit/gerritdialog.cpp
index d3df24049d..f26df525fa 100644
--- a/src/plugins/git/gerrit/gerritdialog.cpp
+++ b/src/plugins/git/gerrit/gerritdialog.cpp
@@ -69,7 +69,7 @@ GerritDialog::GerritDialog(const QSharedPointer<GerritParameters> &p,
     m_ui->remoteComboBox->setParameters(m_parameters);
     m_ui->remoteComboBox->setFallbackEnabled(true);
     m_queryModel->setStringList(m_parameters->savedQueries);
-    QCompleter *completer = new QCompleter(this);
+    auto completer = new QCompleter(this);
     completer->setModel(m_queryModel);
     m_ui->queryLineEdit->setSpecialCompleter(completer);
     m_ui->queryLineEdit->setOkColor(Utils::creatorTheme()->color(Utils::Theme::TextColorNormal));
diff --git a/src/plugins/git/gerrit/gerritdialog.h b/src/plugins/git/gerrit/gerritdialog.h
index 1b15e6f507..98d3015d25 100644
--- a/src/plugins/git/gerrit/gerritdialog.h
+++ b/src/plugins/git/gerrit/gerritdialog.h
@@ -56,7 +56,7 @@ public:
                           const QSharedPointer<GerritServer> &s,
                           const QString &repository,
                           QWidget *parent = nullptr);
-    ~GerritDialog();
+    ~GerritDialog() override;
     QString repositoryPath() const;
     void setCurrentPath(const QString &path);
     void fetchStarted(const QSharedPointer<Gerrit::Internal::GerritChange> &change);
diff --git a/src/plugins/git/gerrit/gerritmodel.cpp b/src/plugins/git/gerrit/gerritmodel.cpp
index 549f33196d..7e504c7a27 100644
--- a/src/plugins/git/gerrit/gerritmodel.cpp
+++ b/src/plugins/git/gerrit/gerritmodel.cpp
@@ -227,7 +227,7 @@ public:
                  const GerritServer &server,
                  QObject *parent = nullptr);
 
-    ~QueryContext();
+    ~QueryContext() override;
     void start();
     void terminate();
 
@@ -845,7 +845,7 @@ QList<QStandardItem *> GerritModel::changeToRow(const GerritChangePtr &c) const
     const QVariant filterV = QVariant(c->filterString());
     const QVariant changeV = qVariantFromValue(c);
     for (int i = 0; i < GerritModel::ColumnCount; ++i) {
-        QStandardItem *item = new QStandardItem;
+        auto item = new QStandardItem;
         item->setData(changeV, GerritModel::GerritChangeRole);
         item->setData(filterV, GerritModel::FilterRole);
         item->setFlags(Qt::ItemIsSelectable | Qt::ItemIsEnabled);
diff --git a/src/plugins/git/gerrit/gerritmodel.h b/src/plugins/git/gerrit/gerritmodel.h
index 213de67db9..c8e89bfae7 100644
--- a/src/plugins/git/gerrit/gerritmodel.h
+++ b/src/plugins/git/gerrit/gerritmodel.h
@@ -105,7 +105,7 @@ public:
         SortRole = Qt::UserRole + 3
     };
     GerritModel(const QSharedPointer<GerritParameters> &, QObject *parent = nullptr);
-    ~GerritModel();
+    ~GerritModel() override;
 
     QVariant data(const QModelIndex &index, int role) const override;
 
diff --git a/src/plugins/git/gerrit/gerritoptionspage.cpp b/src/plugins/git/gerrit/gerritoptionspage.cpp
index 5810649546..730080d66f 100644
--- a/src/plugins/git/gerrit/gerritoptionspage.cpp
+++ b/src/plugins/git/gerrit/gerritoptionspage.cpp
@@ -91,7 +91,7 @@ GerritOptionsWidget::GerritOptionsWidget(QWidget *parent)
     , m_portSpinBox(new QSpinBox(this))
     , m_httpsCheckBox(new QCheckBox(tr("HTTPS")))
 {
-    QFormLayout *formLayout = new QFormLayout(this);
+    auto formLayout = new QFormLayout(this);
     formLayout->setFieldGrowthPolicy(QFormLayout::ExpandingFieldsGrow);
     formLayout->addRow(tr("&Host:"), m_hostLineEdit);
     formLayout->addRow(tr("&User:"), m_userLineEdit);
diff --git a/src/plugins/git/gerrit/gerritoptionspage.h b/src/plugins/git/gerrit/gerritoptionspage.h
index b2a6fce12d..ae2b4776cf 100644
--- a/src/plugins/git/gerrit/gerritoptionspage.h
+++ b/src/plugins/git/gerrit/gerritoptionspage.h
@@ -67,7 +67,7 @@ class GerritOptionsPage : public VcsBase::VcsBaseOptionsPage
 
 public:
     GerritOptionsPage(const QSharedPointer<GerritParameters> &p, QObject *parent = nullptr);
-    ~GerritOptionsPage();
+    ~GerritOptionsPage() override;
 
     QWidget *widget() override;
     void apply() override;
diff --git a/src/plugins/git/gerrit/gerritparameters.cpp b/src/plugins/git/gerrit/gerritparameters.cpp
index b98f36b896..a505bda6e4 100644
--- a/src/plugins/git/gerrit/gerritparameters.cpp
+++ b/src/plugins/git/gerrit/gerritparameters.cpp
@@ -99,8 +99,7 @@ void GerritParameters::setPortFlagBySshType()
 }
 
 GerritParameters::GerritParameters()
-    : https(true)
-    , portFlag(defaultPortFlag)
+    : portFlag(defaultPortFlag)
 {
 }
 
diff --git a/src/plugins/git/gerrit/gerritparameters.h b/src/plugins/git/gerrit/gerritparameters.h
index 7f08f92967..6c5d373990 100644
--- a/src/plugins/git/gerrit/gerritparameters.h
+++ b/src/plugins/git/gerrit/gerritparameters.h
@@ -50,7 +50,7 @@ public:
     QString ssh;
     QString curl;
     QStringList savedQueries;
-    bool https;
+    bool https = true;
     QString portFlag;
 };
 
diff --git a/src/plugins/git/gerrit/gerritplugin.cpp b/src/plugins/git/gerrit/gerritplugin.cpp
index ad87882b38..d409f93906 100644
--- a/src/plugins/git/gerrit/gerritplugin.cpp
+++ b/src/plugins/git/gerrit/gerritplugin.cpp
@@ -94,7 +94,7 @@ public:
                  const QString &repository, const Utils::FileName &git,
                  const GerritServer &server,
                  FetchMode fm, QObject *parent = nullptr);
-    ~FetchContext();
+    ~FetchContext() override;
     void start();
 
 private:
@@ -467,8 +467,7 @@ void GerritPlugin::fetch(const QSharedPointer<GerritChange> &change, int mode)
     if (repository.isEmpty())
         return;
 
-    FetchContext *fc = new FetchContext(change, repository, git,
-                                        *m_server, FetchMode(mode), this);
+    auto fc = new FetchContext(change, repository, git, *m_server, FetchMode(mode), this);
     connect(fc, &QObject::destroyed, this, &GerritPlugin::fetchFinished);
     emit fetchStarted(change);
     fc->start();
diff --git a/src/plugins/git/gerrit/gerritplugin.h b/src/plugins/git/gerrit/gerritplugin.h
index bd9d554bfb..5151235496 100644
--- a/src/plugins/git/gerrit/gerritplugin.h
+++ b/src/plugins/git/gerrit/gerritplugin.h
@@ -53,7 +53,7 @@ class GerritPlugin : public QObject
     Q_OBJECT
 public:
     explicit GerritPlugin(QObject *parent = nullptr);
-    ~GerritPlugin();
+    ~GerritPlugin() override;
 
     bool initialize(Core::ActionContainer *ac);
 
diff --git a/src/plugins/git/gerrit/gerritpushdialog.cpp b/src/plugins/git/gerrit/gerritpushdialog.cpp
index c7e726a4ea..d542c2eb76 100644
--- a/src/plugins/git/gerrit/gerritpushdialog.cpp
+++ b/src/plugins/git/gerrit/gerritpushdialog.cpp
@@ -135,7 +135,7 @@ GerritPushDialog::GerritPushDialog(const QString &workingDir, const QString &rev
     m_ui->remoteComboBox->setParameters(parameters);
     m_ui->remoteComboBox->setAllowDups(true);
 
-    PushItemDelegate *delegate = new PushItemDelegate(m_ui->commitView);
+    auto delegate = new PushItemDelegate(m_ui->commitView);
     delegate->setParent(this);
 
     initRemoteBranches();
diff --git a/src/plugins/git/gerrit/gerritpushdialog.h b/src/plugins/git/gerrit/gerritpushdialog.h
index 9c8254e762..3814fbce3f 100644
--- a/src/plugins/git/gerrit/gerritpushdialog.h
+++ b/src/plugins/git/gerrit/gerritpushdialog.h
@@ -48,7 +48,7 @@ class GerritPushDialog : public QDialog
 public:
     GerritPushDialog(const QString &workingDir, const QString &reviewerList,
                      QSharedPointer<GerritParameters> parameters, QWidget *parent);
-    ~GerritPushDialog();
+    ~GerritPushDialog() override;
 
     QString selectedCommit() const;
     QString selectedRemoteName() const;
diff --git a/src/plugins/git/gitclient.cpp b/src/plugins/git/gitclient.cpp
index d8191b9f6f..b10a6ffe81 100644
--- a/src/plugins/git/gitclient.cpp
+++ b/src/plugins/git/gitclient.cpp
@@ -139,7 +139,7 @@ static QString branchesDisplay(const QString &prefix, QStringList *branches, boo
     //: Displayed after the untranslated message "Branches: branch1, branch2 'and %n more'"
     //  in git show.
     if (more > 0)
-        output += ' ' + GitClient::tr("and %n more", 0, more);
+        output += ' ' + GitClient::tr("and %n more", nullptr, more);
     return output;
 }
 
@@ -187,7 +187,7 @@ bool DescriptionWidgetDecorator::eventFilter(QObject *watched, QEvent *event)
         return QObject::eventFilter(watched, event);
 
     if (event->type() == QEvent::MouseMove) {
-        QMouseEvent *mouseEvent = static_cast<QMouseEvent *>(event);
+        auto mouseEvent = static_cast<QMouseEvent *>(event);
         if (mouseEvent->buttons())
             return QObject::eventFilter(watched, event);
 
@@ -207,7 +207,7 @@ bool DescriptionWidgetDecorator::eventFilter(QObject *watched, QEvent *event)
         textEditor->viewport()->setCursor(cursorShape);
         return ret;
     } else if (event->type() == QEvent::MouseButtonRelease) {
-        QMouseEvent *mouseEvent = static_cast<QMouseEvent *>(event);
+        auto mouseEvent = static_cast<QMouseEvent *>(event);
 
         if (mouseEvent->button() == Qt::LeftButton && !(mouseEvent->modifiers() & Qt::ShiftModifier)) {
             const QTextCursor cursor = textEditor->cursorForPosition(mouseEvent->pos());
@@ -624,12 +624,12 @@ public:
     }
 };
 
-class ConflictHandler : public QObject
+class ConflictHandler final : public QObject
 {
     Q_OBJECT
 public:
     static void attachToCommand(VcsCommand *command, const QString &abortCommand = QString()) {
-        ConflictHandler *handler = new ConflictHandler(command->defaultWorkingDirectory(), abortCommand);
+        auto handler = new ConflictHandler(command->defaultWorkingDirectory(), abortCommand);
         handler->setParent(command); // delete when command goes out of scope
 
         command->addFlags(VcsCommand::ExpectRepoChanges);
@@ -655,7 +655,7 @@ private:
           m_abortCommand(abortCommand)
     { }
 
-    ~ConflictHandler()
+    ~ConflictHandler() final
     {
         // If interactive rebase editor window is closed, plugin is terminated
         // but referenced here when the command ends
@@ -1296,7 +1296,7 @@ bool GitClient::synchronousReset(const QString &workingDirectory,
         if (files.isEmpty()) {
             msgCannotRun(arguments, workingDirectory, resp.stdErr(), errorMessage);
         } else {
-            msgCannotRun(tr("Cannot reset %n files in \"%1\": %2", 0, files.size())
+            msgCannotRun(tr("Cannot reset %n files in \"%1\": %2", nullptr, files.size())
                          .arg(QDir::toNativeSeparators(workingDirectory), resp.stdErr()),
                          errorMessage);
         }
@@ -2632,9 +2632,9 @@ bool GitClient::getCommitData(const QString &workingDirectory,
 static inline QString msgCommitted(const QString &amendSHA1, int fileCount)
 {
     if (amendSHA1.isEmpty())
-        return GitClient::tr("Committed %n files.", 0, fileCount) + '\n';
+        return GitClient::tr("Committed %n files.", nullptr, fileCount) + '\n';
     if (fileCount)
-        return GitClient::tr("Amended \"%1\" (%n files).", 0, fileCount).arg(amendSHA1) + '\n';
+        return GitClient::tr("Amended \"%1\" (%n files).", nullptr, fileCount).arg(amendSHA1) + '\n';
     return GitClient::tr("Amended \"%1\".").arg(amendSHA1);
 }
 
@@ -2725,7 +2725,7 @@ bool GitClient::addAndCommit(const QString &repositoryDirectory,
         VcsOutputWindow::appendError(stdErr);
         return true;
     } else {
-        VcsOutputWindow::appendError(tr("Cannot commit %n files: %1\n", 0, commitCount).arg(stdErr));
+        VcsOutputWindow::appendError(tr("Cannot commit %n files: %1\n", nullptr, commitCount).arg(stdErr));
         return false;
     }
 }
@@ -3241,9 +3241,9 @@ unsigned GitClient::synchronousGitVersion(QString *errorMessage) const
     QRegExp versionPattern("^[^\\d]+(\\d+)\\.(\\d+)\\.(\\d+|rc\\d).*$");
     QTC_ASSERT(versionPattern.isValid(), return 0);
     QTC_ASSERT(versionPattern.exactMatch(output), return 0);
-    const unsigned majorV = versionPattern.cap(1).toUInt(0, 16);
-    const unsigned minorV = versionPattern.cap(2).toUInt(0, 16);
-    const unsigned patchV = versionPattern.cap(3).toUInt(0, 16);
+    const unsigned majorV = versionPattern.cap(1).toUInt(nullptr, 16);
+    const unsigned minorV = versionPattern.cap(2).toUInt(nullptr, 16);
+    const unsigned patchV = versionPattern.cap(3).toUInt(nullptr, 16);
     return version(majorV, minorV, patchV);
 }
 
diff --git a/src/plugins/git/gitgrep.cpp b/src/plugins/git/gitgrep.cpp
index 0940c94036..063deb98a2 100644
--- a/src/plugins/git/gitgrep.cpp
+++ b/src/plugins/git/gitgrep.cpp
@@ -214,7 +214,7 @@ static bool isGitDirectory(const QString &path)
 {
     static IVersionControl *gitVc = VcsManager::versionControl(VcsBase::Constants::VCS_ID_GIT);
     QTC_ASSERT(gitVc, return false);
-    return gitVc == VcsManager::findVersionControlForDirectory(path, 0);
+    return gitVc == VcsManager::findVersionControlForDirectory(path, nullptr);
 }
 
 GitGrep::GitGrep(QObject *parent)
@@ -303,7 +303,8 @@ IEditor *GitGrep::openEditor(const SearchResultItem &item,
     if (content.isEmpty())
         return nullptr;
     QByteArray fileContent;
-    if (TextFileFormat::readFileUTF8(path, 0, &fileContent, 0) == TextFileFormat::ReadSuccess) {
+    if (TextFileFormat::readFileUTF8(path, nullptr, &fileContent, nullptr)
+            == TextFileFormat::ReadSuccess) {
         if (fileContent == content)
             return nullptr; // open the file for read/write
     }
diff --git a/src/plugins/git/githighlighters.cpp b/src/plugins/git/githighlighters.cpp
index d0f87a3cc9..763a3b9f79 100644
--- a/src/plugins/git/githighlighters.cpp
+++ b/src/plugins/git/githighlighters.cpp
@@ -46,7 +46,7 @@ GitSubmitHighlighter::GitSubmitHighlighter(QTextEdit * parent) :
 void GitSubmitHighlighter::highlightBlock(const QString &text)
 {
     // figure out current state
-    State state = static_cast<State>(previousBlockState());
+    auto state = static_cast<State>(previousBlockState());
     if (text.trimmed().isEmpty()) {
         if (state == Header)
             state = Other;
diff --git a/src/plugins/git/githighlighters.h b/src/plugins/git/githighlighters.h
index 055d3721ee..27ab98036e 100644
--- a/src/plugins/git/githighlighters.h
+++ b/src/plugins/git/githighlighters.h
@@ -51,7 +51,7 @@ enum Format {
 class GitSubmitHighlighter : public TextEditor::SyntaxHighlighter
 {
 public:
-    explicit GitSubmitHighlighter(QTextEdit *parent = 0);
+    explicit GitSubmitHighlighter(QTextEdit *parent = nullptr);
     void highlightBlock(const QString &text) override;
 
 private:
@@ -65,7 +65,7 @@ private:
 class GitRebaseHighlighter : public TextEditor::SyntaxHighlighter
 {
 public:
-    explicit GitRebaseHighlighter(QTextDocument *parent = 0);
+    explicit GitRebaseHighlighter(QTextDocument *parent = nullptr);
     void highlightBlock(const QString &text) override;
 
 private:
diff --git a/src/plugins/git/gitplugin.cpp b/src/plugins/git/gitplugin.cpp
index a71ace6e8c..2869d73bc7 100644
--- a/src/plugins/git/gitplugin.cpp
+++ b/src/plugins/git/gitplugin.cpp
@@ -128,7 +128,7 @@ const VcsBaseEditorParameters editorParameters[] = {
 
 // GitPlugin
 
-static GitPlugin *m_instance = 0;
+static GitPlugin *m_instance = nullptr;
 
 GitPlugin::GitPlugin()
 {
@@ -142,7 +142,7 @@ GitPlugin::~GitPlugin()
 {
     cleanCommitMessageFile();
     delete m_gitClient;
-    m_instance = 0;
+    m_instance = nullptr;
 }
 
 void GitPlugin::cleanCommitMessageFile()
@@ -986,8 +986,8 @@ void GitPlugin::updateVersionWarning()
 IEditor *GitPlugin::openSubmitEditor(const QString &fileName, const CommitData &cd)
 {
     IEditor *editor = EditorManager::openEditor(fileName, Constants::GITSUBMITEDITOR_ID);
-    GitSubmitEditor *submitEditor = qobject_cast<GitSubmitEditor*>(editor);
-    QTC_ASSERT(submitEditor, return 0);
+    auto submitEditor = qobject_cast<GitSubmitEditor*>(editor);
+    QTC_ASSERT(submitEditor, return nullptr);
     setSubmitEditor(submitEditor);
     submitEditor->setCommitData(cd);
     submitEditor->setCheckScriptWorkingDirectory(m_submitRepository);
@@ -1020,7 +1020,7 @@ bool GitPlugin::submitEditorAboutToClose()
 {
     if (!isCommitEditorOpen())
         return true;
-    GitSubmitEditor *editor = qobject_cast<GitSubmitEditor *>(submitEditor());
+    auto editor = qobject_cast<GitSubmitEditor *>(submitEditor());
     QTC_ASSERT(editor, return true);
     IDocument *editorDocument = editor->document();
     QTC_ASSERT(editorDocument, return true);
@@ -1052,7 +1052,7 @@ bool GitPlugin::submitEditorAboutToClose()
 
 
     // Go ahead!
-    SubmitFileModel *model = qobject_cast<SubmitFileModel *>(editor->fileModel());
+    auto model = qobject_cast<SubmitFileModel *>(editor->fileModel());
     CommitType commitType = editor->commitType();
     QString amendSHA1 = editor->amendSHA1();
     if (model->hasCheckedFiles() || !amendSHA1.isEmpty()) {
diff --git a/src/plugins/git/gitplugin.h b/src/plugins/git/gitplugin.h
index a4474379e3..3a73ebc1b7 100644
--- a/src/plugins/git/gitplugin.h
+++ b/src/plugins/git/gitplugin.h
@@ -74,7 +74,7 @@ class GitPlugin : public VcsBase::VcsBasePlugin
 
 public:
     GitPlugin();
-    ~GitPlugin();
+    ~GitPlugin() override;
 
     static GitPlugin *instance();
     static GitClient *client();
diff --git a/src/plugins/git/gitsubmiteditor.cpp b/src/plugins/git/gitsubmiteditor.cpp
index 4263219e8e..cca31f8724 100644
--- a/src/plugins/git/gitsubmiteditor.cpp
+++ b/src/plugins/git/gitsubmiteditor.cpp
@@ -51,13 +51,13 @@ namespace Internal {
 class GitSubmitFileModel : public SubmitFileModel
 {
 public:
-    GitSubmitFileModel(QObject *parent = 0) : SubmitFileModel(parent)
+    GitSubmitFileModel(QObject *parent = nullptr) : SubmitFileModel(parent)
     { }
 
     void updateSelections(SubmitFileModel *source) override
     {
         QTC_ASSERT(source, return);
-        GitSubmitFileModel *gitSource = static_cast<GitSubmitFileModel *>(source);
+        auto gitSource = static_cast<GitSubmitFileModel *>(source);
         int j = 0;
         for (int i = 0; i < rowCount() && j < source->rowCount(); ++i) {
             CommitData::StateFilePair stateFile = stateFilePair(i);
diff --git a/src/plugins/git/gitutils.cpp b/src/plugins/git/gitutils.cpp
index 371a00885e..c0f5ffd754 100644
--- a/src/plugins/git/gitutils.cpp
+++ b/src/plugins/git/gitutils.cpp
@@ -74,7 +74,7 @@ bool inputText(QWidget *parent, const QString &title, const QString &prompt, QSt
     dialog.setLabelText(prompt);
     dialog.setTextValue(*s);
     // Nasty hack:
-    if (QLineEdit *le = dialog.findChild<QLineEdit*>())
+    if (auto le = dialog.findChild<QLineEdit*>())
         le->setMinimumWidth(500);
     if (dialog.exec() != QDialog::Accepted)
         return false;
diff --git a/src/plugins/git/logchangedialog.cpp b/src/plugins/git/logchangedialog.cpp
index c6dd0a4a16..e57dda49e0 100644
--- a/src/plugins/git/logchangedialog.cpp
+++ b/src/plugins/git/logchangedialog.cpp
@@ -159,7 +159,7 @@ bool LogChangeWidget::populateLog(const QString &repository, const QString &comm
         arguments << "--not" << "--remotes";
     arguments << "--";
     QString output;
-    if (!GitPlugin::client()->synchronousLog(repository, arguments, &output, 0, VcsCommand::NoOutput))
+    if (!GitPlugin::client()->synchronousLog(repository, arguments, &output, nullptr, VcsCommand::NoOutput))
         return false;
     const QStringList lines = output.split('\n');
     for (const QString &line : lines) {
@@ -193,14 +193,13 @@ const QStandardItem *LogChangeWidget::currentItem(int column) const
     const QModelIndex currentIndex = selectionModel()->currentIndex();
     if (currentIndex.isValid())
         return m_model->item(currentIndex.row(), column);
-    return 0;
+    return nullptr;
 }
 
 LogChangeDialog::LogChangeDialog(bool isReset, QWidget *parent) :
     QDialog(parent)
     , m_widget(new LogChangeWidget)
     , m_dialogButtonBox(new QDialogButtonBox(this))
-    , m_resetTypeComboBox(0)
 {
     auto layout = new QVBoxLayout(this);
     layout->addWidget(new QLabel(isReset ? tr("Reset to:") : tr("Select change:"), this));
diff --git a/src/plugins/git/logchangedialog.h b/src/plugins/git/logchangedialog.h
index 9576d9d361..dfd83f4bf0 100644
--- a/src/plugins/git/logchangedialog.h
+++ b/src/plugins/git/logchangedialog.h
@@ -96,9 +96,9 @@ public:
     LogChangeWidget *widget() const;
 
 private:
-    LogChangeWidget *m_widget;
-    QDialogButtonBox *m_dialogButtonBox;
-    QComboBox *m_resetTypeComboBox;
+    LogChangeWidget *m_widget = nullptr;
+    QDialogButtonBox *m_dialogButtonBox = nullptr;
+    QComboBox *m_resetTypeComboBox = nullptr;
 };
 
 class LogItemDelegate : public QStyledItemDelegate
diff --git a/src/plugins/git/mergetool.h b/src/plugins/git/mergetool.h
index d5d3d45f64..667ba09fdb 100644
--- a/src/plugins/git/mergetool.h
+++ b/src/plugins/git/mergetool.h
@@ -50,8 +50,8 @@ class MergeTool : public QObject
     };
 
 public:
-    explicit MergeTool(QObject *parent = 0);
-    ~MergeTool();
+    explicit MergeTool(QObject *parent = nullptr);
+    ~MergeTool() override;
     bool start(const QString &workingDirectory, const QStringList &files = QStringList());
 
     enum MergeType {
diff --git a/src/plugins/git/remotedialog.h b/src/plugins/git/remotedialog.h
index 8495fdc73a..1026c2e721 100644
--- a/src/plugins/git/remotedialog.h
+++ b/src/plugins/git/remotedialog.h
@@ -39,7 +39,7 @@ class RemoteDialog : public QDialog
     Q_OBJECT
 
 public:
-    explicit RemoteDialog(QWidget *parent = 0);
+    explicit RemoteDialog(QWidget *parent = nullptr);
     ~RemoteDialog() override;
 
     void refresh(const QString &repository, bool force);
diff --git a/src/plugins/git/remotemodel.h b/src/plugins/git/remotemodel.h
index 6ac83e32fa..d0622996a0 100644
--- a/src/plugins/git/remotemodel.h
+++ b/src/plugins/git/remotemodel.h
@@ -35,7 +35,7 @@ namespace Internal {
 class RemoteModel : public QAbstractTableModel {
     Q_OBJECT
 public:
-    explicit RemoteModel(QObject *parent = 0);
+    explicit RemoteModel(QObject *parent = nullptr);
 
     void clear();
     bool refresh(const QString &workingDirectory, QString *errorMessage);
diff --git a/src/plugins/git/settingspage.cpp b/src/plugins/git/settingspage.cpp
index 123d4d2938..9026d0f26f 100644
--- a/src/plugins/git/settingspage.cpp
+++ b/src/plugins/git/settingspage.cpp
@@ -120,7 +120,7 @@ void SettingsPage::apply()
 
     if (widget()->isVisible()) {
         const VcsBaseClientSettings settings = widget()->settings();
-        const GitSettings *rc = static_cast<const GitSettings *>(&settings);
+        auto rc = static_cast<const GitSettings *>(&settings);
         bool gitFoundOk;
         QString errorMessage;
         rc->gitExecutable(&gitFoundOk, &errorMessage);
diff --git a/src/plugins/git/settingspage.h b/src/plugins/git/settingspage.h
index 187cf75af9..e97ae59dec 100644
--- a/src/plugins/git/settingspage.h
+++ b/src/plugins/git/settingspage.h
@@ -43,7 +43,7 @@ class SettingsPageWidget : public VcsBase::VcsClientOptionsPageWidget {
     Q_OBJECT
 
 public:
-    explicit SettingsPageWidget(QWidget *parent = 0);
+    explicit SettingsPageWidget(QWidget *parent = nullptr);
 
     VcsBase::VcsBaseClientSettings settings() const override;
     void setSettings(const VcsBase::VcsBaseClientSettings &s) override;
diff --git a/src/plugins/git/stashdialog.cpp b/src/plugins/git/stashdialog.cpp
index ffa9ac6624..e20cb6dcc8 100644
--- a/src/plugins/git/stashdialog.cpp
+++ b/src/plugins/git/stashdialog.cpp
@@ -63,7 +63,7 @@ static inline QList<QStandardItem*> stashModelRowItems(const Stash &s)
 // -----------  StashModel
 class StashModel : public QStandardItemModel {
 public:
-    explicit StashModel(QObject *parent = 0);
+    explicit StashModel(QObject *parent = nullptr);
 
     void setStashes(const QList<Stash> &stashes);
     const Stash &at(int i) { return m_stashes.at(i); }
@@ -189,7 +189,7 @@ void StashDialog::deleteSelection()
     const QList<int> rows = selectedRows();
     QTC_ASSERT(!rows.isEmpty(), return);
     const QString title = tr("Delete Stashes");
-    if (!ask(title, tr("Do you want to delete %n stash(es)?", 0, rows.size())))
+    if (!ask(title, tr("Do you want to delete %n stash(es)?", nullptr, rows.size())))
         return;
     QString errorMessage;
     QStringList errors;
@@ -267,7 +267,7 @@ bool StashDialog::promptForRestore(QString *stash,
 {
     const QString stashIn = *stash;
     bool modifiedPromptShown = false;
-    switch (GitPlugin::client()->gitStatus(m_repository, StatusMode(NoUntracked | NoSubmodules), 0, errorMessage)) {
+    switch (GitPlugin::client()->gitStatus(m_repository, StatusMode(NoUntracked | NoSubmodules), nullptr, errorMessage)) {
     case GitClient::StatusFailed:
         return false;
     case GitClient::StatusChanged: {
@@ -317,7 +317,7 @@ void StashDialog::restoreCurrent()
     QString name = m_model->at(index).name;
     // Make sure repository is not modified, restore. The command will
     // output to window on success.
-    if (promptForRestore(&name, 0, &errorMessage)
+    if (promptForRestore(&name, nullptr, &errorMessage)
             && GitPlugin::client()->synchronousStashRestore(m_repository, name)) {
         refresh(m_repository, true); // Might have stashed away local changes.
     } else if (!errorMessage.isEmpty()) {
diff --git a/src/plugins/git/stashdialog.h b/src/plugins/git/stashdialog.h
index bf72509094..5aa8c52213 100644
--- a/src/plugins/git/stashdialog.h
+++ b/src/plugins/git/stashdialog.h
@@ -48,7 +48,7 @@ class StashDialog : public QDialog
     Q_OBJECT
 
 public:
-    explicit StashDialog(QWidget *parent = 0);
+    explicit StashDialog(QWidget *parent = nullptr);
     ~StashDialog() override;
 
     void refresh(const QString &repository, bool force);