feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

Android: Get rid of onLicenseStdOut()

Browse Source 
Make code more readable.

Change-Id: Icc01a90d7f1469bb58fba325bca728e002c79512
Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
Reviewed-by: Alessandro Portale <alessandro.portale@qt.io>
This commit is contained in:
Jarek Kobus
2024-04-15 23:47:37 +02:00
parent b8e1b33455
commit bd282f1ada
1 changed files with 9 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
src/plugins/android/androidsdkmanager.cpp
View File

@@ -4,7 +4,6 @@
#include "androidconfigurations.h"
#include "androidsdkmanager.h"
#include "androidtr.h"
#include "avdmanageroutputparser.h"
#include "sdkmanageroutputparser.h"
#include <utils/algorithm.h>
@@ -46,18 +45,6 @@ static const QRegularExpression &assertionRegExp()
return theRegExp;
}
static std::optional<QString> onLicenseStdOut(const QString &output, QString *licenseTextCache)
{
licenseTextCache->append(output);
const QRegularExpressionMatch assertionMatch = assertionRegExp().match(*licenseTextCache);
if (assertionMatch.hasMatch()) {
const QString ret = *licenseTextCache;
licenseTextCache->clear();
return ret;
}
return {};
}
int parseProgress(const QString &out, bool &foundAssertion)
{
int progress = -1;
@@ -540,12 +527,16 @@ void AndroidSdkManagerPrivate::getPendingLicense(SdkCmdPromise &fi)
QString licenseTextCache;
while (!licenseCommand.waitForFinished(200ms)) {
const QString stdOut = codec->toUnicode(licenseCommand.readAllRawStandardOutput());
std::optional<QString> assertion;
bool assertion = false;
if (!stdOut.isEmpty()) {
assertion = onLicenseStdOut(stdOut, &licenseTextCache);
if (assertion && reviewingLicenses) {
result.stdOutput = *assertion;
fi.addResult(result);
licenseTextCache.append(stdOut);
assertion = assertionRegExp().match(licenseTextCache).hasMatch();
if (assertion) {
if (reviewingLicenses) {
result.stdOutput = licenseTextCache;
fi.addResult(result);
}
licenseTextCache.clear();
}
}