diff --git a/src/plugins/bazaar/annotationhighlighter.cpp b/src/plugins/bazaar/annotationhighlighter.cpp index 8f99f9ec06a..7b5e2ba0208 100644 --- a/src/plugins/bazaar/annotationhighlighter.cpp +++ b/src/plugins/bazaar/annotationhighlighter.cpp @@ -19,7 +19,7 @@ QString BazaarAnnotationHighlighter::changeNumber(const QString &block) const const QRegularExpressionMatch match = m_changeset.match(block); if (match.hasMatch()) return match.captured(1); - return QString(); + return {}; } } // Bazaar::Internal diff --git a/src/plugins/bazaar/bazaarcommitwidget.cpp b/src/plugins/bazaar/bazaarcommitwidget.cpp index 379fca66610..465395e0aef 100644 --- a/src/plugins/bazaar/bazaarcommitwidget.cpp +++ b/src/plugins/bazaar/bazaarcommitwidget.cpp @@ -159,7 +159,7 @@ QString BazaarCommitWidget::committer() const const QString author = m_bazaarCommitPanel->authorLineEdit->text(); const QString email = m_bazaarCommitPanel->emailLineEdit->text(); if (author.isEmpty()) - return QString(); + return {}; QString user = author; if (!email.isEmpty()) { diff --git a/src/plugins/bazaar/bazaareditor.cpp b/src/plugins/bazaar/bazaareditor.cpp index 956ed6f28a9..146b2e71932 100644 --- a/src/plugins/bazaar/bazaareditor.cpp +++ b/src/plugins/bazaar/bazaareditor.cpp @@ -56,7 +56,7 @@ QString BazaarEditorWidget::changeUnderCursor(const QTextCursor &cursorIn) const } } } - return QString(); + return {}; } VcsBase::BaseAnnotationHighlighter *BazaarEditorWidget::createAnnotationHighlighter(const QSet &changes) const diff --git a/src/plugins/bazaar/pullorpushdialog.cpp b/src/plugins/bazaar/pullorpushdialog.cpp index 4c2cd99e7c5..1b607622369 100644 --- a/src/plugins/bazaar/pullorpushdialog.cpp +++ b/src/plugins/bazaar/pullorpushdialog.cpp @@ -112,7 +112,7 @@ PullOrPushDialog::~PullOrPushDialog() = default; QString PullOrPushDialog::branchLocation() const { if (m_defaultButton->isChecked()) - return QString(); + return {}; if (m_localButton->isChecked()) return m_localPathChooser->filePath().toString(); return m_urlLineEdit->text(); diff --git a/src/plugins/cvs/cvseditor.cpp b/src/plugins/cvs/cvseditor.cpp index 0a9298bd47e..d28144e3146 100644 --- a/src/plugins/cvs/cvseditor.cpp +++ b/src/plugins/cvs/cvseditor.cpp @@ -90,7 +90,7 @@ QString CvsEditorWidget::changeUnderCursor(const QTextCursor &c) const } break; } - return QString(); + return {}; } VcsBase::BaseAnnotationHighlighter *CvsEditorWidget::createAnnotationHighlighter(const QSet &changes) const diff --git a/src/plugins/cvs/cvssubmiteditor.cpp b/src/plugins/cvs/cvssubmiteditor.cpp index d5ecf9fcf4c..983dadaee32 100644 --- a/src/plugins/cvs/cvssubmiteditor.cpp +++ b/src/plugins/cvs/cvssubmiteditor.cpp @@ -29,7 +29,7 @@ QString CvsSubmitEditor::stateName(State st) const case LocallyRemoved: return m_msgRemoved; } - return QString(); + return {}; } void CvsSubmitEditor::setStateList(const StateFilePairs &statusOutput) diff --git a/src/plugins/fossil/pullorpushdialog.cpp b/src/plugins/fossil/pullorpushdialog.cpp index b6473a68b41..8c0df1dd522 100644 --- a/src/plugins/fossil/pullorpushdialog.cpp +++ b/src/plugins/fossil/pullorpushdialog.cpp @@ -82,7 +82,7 @@ PullOrPushDialog::PullOrPushDialog(Mode mode, QWidget *parent) QString PullOrPushDialog::remoteLocation() const { if (m_defaultButton->isChecked()) - return QString(); + return {}; if (m_localButton->isChecked()) return m_localPathChooser->filePath().toString(); return m_urlLineEdit->text(); diff --git a/src/plugins/fossil/wizard/fossiljsextension.cpp b/src/plugins/fossil/wizard/fossiljsextension.cpp index 9ca6b6e0bf0..aaf5d172b67 100644 --- a/src/plugins/fossil/wizard/fossiljsextension.cpp +++ b/src/plugins/fossil/wizard/fossiljsextension.cpp @@ -47,26 +47,17 @@ QString FossilJsExtension::displayName() const QString FossilJsExtension::defaultAdminUser() const { - if (!isConfigured()) - return QString(); - - return settings().userName(); + return isConfigured() ? settings().userName() : QString(); } QString FossilJsExtension::defaultSslIdentityFile() const { - if (!isConfigured()) - return QString(); - - return settings().sslIdentityFile().toFSPathString(); + return isConfigured() ? settings().sslIdentityFile().toFSPathString() : QString(); } QString FossilJsExtension::defaultLocalRepoPath() const { - if (!isConfigured()) - return QString(); - - return settings().defaultRepoPath().toFSPathString(); + return isConfigured() ? settings().defaultRepoPath().toFSPathString() : QString(); } bool FossilJsExtension::defaultDisableAutosync() const diff --git a/src/plugins/mercurial/annotationhighlighter.cpp b/src/plugins/mercurial/annotationhighlighter.cpp index be2259182f0..ec75322a71e 100644 --- a/src/plugins/mercurial/annotationhighlighter.cpp +++ b/src/plugins/mercurial/annotationhighlighter.cpp @@ -18,7 +18,7 @@ QString MercurialAnnotationHighlighter::changeNumber(const QString &block) const const QRegularExpressionMatch match = changeset.match(block); if (match.hasMatch()) return match.captured(1); - return QString(); + return {}; } } // Mercurial::Internal diff --git a/src/plugins/mercurial/mercurialcommitwidget.cpp b/src/plugins/mercurial/mercurialcommitwidget.cpp index c6b35a67d08..d1d1a7b37b9 100644 --- a/src/plugins/mercurial/mercurialcommitwidget.cpp +++ b/src/plugins/mercurial/mercurialcommitwidget.cpp @@ -151,7 +151,7 @@ QString MercurialCommitWidget::committer() const const QString author = mercurialCommitPanel->m_authorLineEdit->text(); const QString email = mercurialCommitPanel->m_emailLineEdit->text(); if (author.isEmpty()) - return QString(); + return {}; QString user = author; if (!email.isEmpty()) { diff --git a/src/plugins/mercurial/mercurialeditor.cpp b/src/plugins/mercurial/mercurialeditor.cpp index 19e1921e02c..b08802b8d66 100644 --- a/src/plugins/mercurial/mercurialeditor.cpp +++ b/src/plugins/mercurial/mercurialeditor.cpp @@ -45,7 +45,7 @@ QString MercurialEditorWidget::changeUnderCursor(const QTextCursor &cursorIn) co if (exactIdentifier40.match(change).hasMatch()) return change; } - return QString(); + return {}; } VcsBase::BaseAnnotationHighlighter *MercurialEditorWidget::createAnnotationHighlighter(const QSet &changes) const diff --git a/src/plugins/perforce/perforceeditor.cpp b/src/plugins/perforce/perforceeditor.cpp index ec418a7cedd..623ac1ed8fe 100644 --- a/src/plugins/perforce/perforceeditor.cpp +++ b/src/plugins/perforce/perforceeditor.cpp @@ -46,7 +46,7 @@ QString PerforceEditorWidget::changeUnderCursor(const QTextCursor &c) const // Any number is regarded as change number. cursor.select(QTextCursor::WordUnderCursor); if (!cursor.hasSelection()) - return QString(); + return {}; const QString change = cursor.selectedText(); return m_changeNumberPattern.match(change).hasMatch() ? change : QString(); } diff --git a/src/plugins/perforce/perforceplugin.cpp b/src/plugins/perforce/perforceplugin.cpp index 2a9905a4c22..64ad692c889 100644 --- a/src/plugins/perforce/perforceplugin.cpp +++ b/src/plugins/perforce/perforceplugin.cpp @@ -1589,7 +1589,7 @@ QString PerforcePluginPrivate::clientFilePath(const QString &serverFilePath) const PerforceResponse response = runP4Cmd(m_settings.topLevelSymLinkTarget(), args, ShowBusyCursor|RunFullySynchronous|CommandToWindow|StdErrToWindow|ErrorToWindow); if (response.error) - return QString(); + return {}; const QRegularExpression r("\\.\\.\\.\\sclientFile\\s(.+?)\n"); const QRegularExpressionMatch match = r.match(response.stdOut); @@ -1604,14 +1604,14 @@ QString PerforcePluginPrivate::pendingChangesData() const PerforceResponse userResponse = runP4Cmd(m_settings.topLevelSymLinkTarget(), args, RunFullySynchronous|CommandToWindow|StdErrToWindow|ErrorToWindow); if (userResponse.error) - return QString(); + return {}; const QRegularExpression r("User\\sname:\\s(\\S+?)\\s*?\n"); QTC_ASSERT(r.isValid(), return QString()); const QRegularExpressionMatch match = r.match(userResponse.stdOut); const QString user = match.hasMatch() ? match.captured(1).trimmed() : QString(); if (user.isEmpty()) - return QString(); + return {}; args.clear(); args << QLatin1String("changes") << QLatin1String("-s") << QLatin1String("pending") << QLatin1String("-u") << user; const PerforceResponse dataResponse = runP4Cmd(m_settings.topLevelSymLinkTarget(), args, @@ -1644,7 +1644,7 @@ QString PerforcePlugin::fileNameFromPerforceName(const QString& perforceName, const PerforceResponse response = dd->runP4Cmd(dd->m_settings.topLevelSymLinkTarget(), args, flags); if (response.error) { *errorMessage = msgWhereFailed(perforceName, response.message); - return QString(); + return {}; } QString output = response.stdOut; @@ -1656,7 +1656,7 @@ QString PerforcePlugin::fileNameFromPerforceName(const QString& perforceName, if (output.isEmpty()) { //: File is not managed by Perforce *errorMessage = msgWhereFailed(perforceName, Tr::tr("The file is not mapped")); - return QString(); + return {}; } const QString p4fileSpec = output.mid(output.lastIndexOf(QLatin1Char(' ')) + 1); return dd->m_settings.mapToFileSystem(p4fileSpec); diff --git a/src/plugins/subversion/subversionclient.cpp b/src/plugins/subversion/subversionclient.cpp index 0e47c584542..5bb2a3eac2f 100644 --- a/src/plugins/subversion/subversionclient.cpp +++ b/src/plugins/subversion/subversionclient.cpp @@ -89,7 +89,7 @@ Id SubversionClient::vcsEditorKind(VcsCommandTag cmd) const case VcsBaseClient::LogCommand: return Constants::SUBVERSION_LOG_EDITOR_ID; case VcsBaseClient::AnnotateCommand: return Constants::SUBVERSION_BLAME_EDITOR_ID; default: - return Id(); + return {}; } } diff --git a/src/plugins/subversion/subversioneditor.cpp b/src/plugins/subversion/subversioneditor.cpp index c54371c7867..4549c6e14da 100644 --- a/src/plugins/subversion/subversioneditor.cpp +++ b/src/plugins/subversion/subversioneditor.cpp @@ -46,7 +46,7 @@ QString SubversionEditorWidget::changeUnderCursor(const QTextCursor &c) const // Any number is regarded as change number. cursor.select(QTextCursor::LineUnderCursor); if (!cursor.hasSelection()) - return QString(); + return {}; const QString change = cursor.selectedText(); const int pos = c.position() - cursor.selectionStart() + 1; // Annotation output has number, log output has revision numbers, @@ -72,7 +72,7 @@ QString SubversionEditorWidget::changeUnderCursor(const QTextCursor &c) const if (pos > start && pos <= end) return rev; } - return QString(); + return {}; } VcsBase::BaseAnnotationHighlighter *SubversionEditorWidget::createAnnotationHighlighter(const QSet &changes) const diff --git a/src/plugins/subversion/subversionplugin.cpp b/src/plugins/subversion/subversionplugin.cpp index 7ddcb0ddf6c..fb6fca48d4f 100644 --- a/src/plugins/subversion/subversionplugin.cpp +++ b/src/plugins/subversion/subversionplugin.cpp @@ -994,7 +994,7 @@ QString SubversionPluginPrivate::monitorFile(const FilePath &repository) const return fi.absoluteFilePath(); } } - return QString(); + return {}; } QString SubversionPluginPrivate::synchronousTopic(const FilePath &repository) const