QtcProcess: Small cleanup

Change-Id: Ib3c913185b01ad3f81e883bd3013ccb95a863485
Reviewed-by: hjk <hjk@qt.io>
This commit is contained in:
Jarek Kobus
2022-10-06 17:06:36 +02:00
parent fa63c68fd9
commit b2ee86ba03

View File

@@ -126,7 +126,7 @@ private:
hitThisMainField, totalThisMainField, hitAllMainField, totalAllMainField); hitThisMainField, totalThisMainField, hitAllMainField, totalAllMainField);
qDebug("%s", qPrintable(totalString)); qDebug("%s", qPrintable(totalString));
} }
static QString formatField(int number, int fieldWidth, const QString &suffix = QString()) static QString formatField(int number, int fieldWidth, const QString &suffix = {})
{ {
return QString("%1%2").arg(number, fieldWidth - suffix.count()).arg(suffix); return QString("%1%2").arg(number, fieldWidth - suffix.count()).arg(suffix);
} }
@@ -1338,10 +1338,10 @@ static QString checkBinary(const QDir &dir, const QString &binary)
// Does the OS have some weird extension concept or does the // Does the OS have some weird extension concept or does the
// binary have a 3 letter extension? // binary have a 3 letter extension?
if (HostOsInfo::isAnyUnixHost() && !HostOsInfo::isMacHost()) if (HostOsInfo::isAnyUnixHost() && !HostOsInfo::isMacHost())
return QString(); return {};
const int dotIndex = binary.lastIndexOf(QLatin1Char('.')); const int dotIndex = binary.lastIndexOf(QLatin1Char('.'));
if (dotIndex != -1 && dotIndex == binary.size() - 4) if (dotIndex != -1 && dotIndex == binary.size() - 4)
return QString(); return {};
switch (HostOsInfo::hostOs()) { switch (HostOsInfo::hostOs()) {
case OsTypeLinux: case OsTypeLinux:
@@ -1373,7 +1373,7 @@ static QString checkBinary(const QDir &dir, const QString &binary)
} }
break; break;
} }
return QString(); return {};
} }
QString QtcProcess::locateBinary(const QString &path, const QString &binary) QString QtcProcess::locateBinary(const QString &path, const QString &binary)
@@ -1392,7 +1392,7 @@ QString QtcProcess::locateBinary(const QString &path, const QString &binary)
const QStringList paths = path.split(HostOsInfo::pathListSeparator()); const QStringList paths = path.split(HostOsInfo::pathListSeparator());
if (paths.empty()) if (paths.empty())
return QString(); return {};
const QStringList::const_iterator cend = paths.constEnd(); const QStringList::const_iterator cend = paths.constEnd();
for (QStringList::const_iterator it = paths.constBegin(); it != cend; ++it) { for (QStringList::const_iterator it = paths.constBegin(); it != cend; ++it) {
const QDir dir(*it); const QDir dir(*it);
@@ -1400,7 +1400,7 @@ QString QtcProcess::locateBinary(const QString &path, const QString &binary)
if (!rc.isEmpty()) if (!rc.isEmpty())
return rc; return rc;
} }
return QString(); return {};
} }
Environment QtcProcess::systemEnvironmentForBinary(const FilePath &filePath) Environment QtcProcess::systemEnvironmentForBinary(const FilePath &filePath)
@@ -1584,7 +1584,7 @@ QString QtcProcess::exitMessage() const
return QtcProcess::tr("The command \"%1\" did not respond within the timeout limit (%2 s).") return QtcProcess::tr("The command \"%1\" did not respond within the timeout limit (%2 s).")
.arg(fullCmd).arg(d->m_maxHangTimerCount); .arg(fullCmd).arg(d->m_maxHangTimerCount);
} }
return QString(); return {};
} }
QByteArray QtcProcess::allRawOutput() const QByteArray QtcProcess::allRawOutput() const