Port androiddeployqt and uic from QStringRef to QStringView

Task-number: QTBUG-84319
Change-Id: I7f30a4d73dd98ee1977645d7274816cd71307506
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
This commit is contained in:
Lars Knoll 2020-05-27 11:02:35 +02:00
parent 31828b3d6b
commit 2393432cd0
5 changed files with 14 additions and 13 deletions

View File

@ -983,9 +983,10 @@ bool readInputFile(Options *options)
const QJsonValue deploymentDependencies = jsonObject.value(QLatin1String("deployment-dependencies")); const QJsonValue deploymentDependencies = jsonObject.value(QLatin1String("deployment-dependencies"));
if (!deploymentDependencies.isUndefined()) { if (!deploymentDependencies.isUndefined()) {
QString deploymentDependenciesString = deploymentDependencies.toString(); QString deploymentDependenciesString = deploymentDependencies.toString();
const auto dependencies = deploymentDependenciesString.splitRef(QLatin1Char(',')); const auto dependencies = QStringView{deploymentDependenciesString}.split(QLatin1Char(','));
for (const QStringRef &dependency : dependencies) { for (const auto &dependency : dependencies) {
QString path = options->qtInstallDirectory + QLatin1Char('/') + dependency; QString path = options->qtInstallDirectory + QChar::fromLatin1('/');
path += dependency;
if (QFileInfo(path).isDir()) { if (QFileInfo(path).isDir()) {
QDirIterator iterator(path, QDirIterator::Subdirectories); QDirIterator iterator(path, QDirIterator::Subdirectories);
while (iterator.hasNext()) { while (iterator.hasNext()) {
@ -1137,7 +1138,7 @@ bool copyAndroidExtraLibs(Options *options)
return true; return true;
if (options->verbose) if (options->verbose)
fprintf(stdout, "Copying %zd external libraries to package.\n", options->extraLibs.size()); fprintf(stdout, "Copying %zd external libraries to package.\n", qsizetype(options->extraLibs.size()));
for (const QString &extraLib : options->extraLibs) { for (const QString &extraLib : options->extraLibs) {
QFileInfo extraLibInfo(extraLib); QFileInfo extraLibInfo(extraLib);

View File

@ -315,7 +315,7 @@ void WriteIncludes::writeHeaders(const OrderedSet &headers, bool global)
// Check for the old headers 'qslider.h' and replace by 'QtGui/QSlider' // Check for the old headers 'qslider.h' and replace by 'QtGui/QSlider'
for (const QString &header : headers) { for (const QString &header : headers) {
const QString value = m_oldHeaderToNewHeader.value(header, header); const QString value = m_oldHeaderToNewHeader.value(header, header);
const auto trimmed = QStringRef(&value).trimmed(); const auto trimmed = QStringView(value).trimmed();
if (!trimmed.isEmpty()) if (!trimmed.isEmpty())
m_output << "#include " << openingQuote << trimmed << closingQuote << '\n'; m_output << "#include " << openingQuote << trimmed << closingQuote << '\n';
} }

View File

@ -1308,7 +1308,7 @@ void WriteInitialization::writeProperties(const QString &varName,
QTextStream str(&setFunction); QTextStream str(&setFunction);
if (stdset) { if (stdset) {
str << language::derefPointer <<"set" << propertyName.at(0).toUpper() str << language::derefPointer <<"set" << propertyName.at(0).toUpper()
<< propertyName.midRef(1) << '('; << QStringView{propertyName}.mid(1) << '(';
} else { } else {
str << language::derefPointer << QLatin1String("setProperty(\"") str << language::derefPointer << QLatin1String("setProperty(\"")
<< propertyName << "\", "; << propertyName << "\", ";
@ -2156,7 +2156,7 @@ void WriteInitialization::addInitializer(Item *item,
if (!value.isEmpty()) { if (!value.isEmpty()) {
QString setter; QString setter;
QTextStream str(&setter); QTextStream str(&setter);
str << language::derefPointer << "set" << name.at(0).toUpper() << name.midRef(1) << '('; str << language::derefPointer << "set" << name.at(0).toUpper() << QStringView{name}.mid(1) << '(';
if (column >= 0) if (column >= 0)
str << column << ", "; str << column << ", ";
str << value << ");"; str << value << ");";

View File

@ -394,12 +394,12 @@ static void formatMemberFnPtr(QTextStream &str, const SignalSlot &s,
const int parenPos = s.signature.indexOf(QLatin1Char('(')); const int parenPos = s.signature.indexOf(QLatin1Char('('));
Q_ASSERT(parenPos >= 0); Q_ASSERT(parenPos >= 0);
if (useQOverload) { if (useQOverload) {
const auto parameters = s.signature.midRef(parenPos + 1, const auto parameters = QStringView{s.signature}.mid(parenPos + 1,
s.signature.size() - parenPos - 2); s.signature.size() - parenPos - 2);
str << "qOverload<" << parameters << ">("; str << "qOverload<" << parameters << ">(";
} }
const auto functionName = s.signature.leftRef(parenPos); const auto functionName = QStringView{s.signature}.left(parenPos);
str << '&' << s.className << "::" << functionName; str << '&' << s.className << "::" << functionName;
if (useQOverload) if (useQOverload)
@ -441,9 +441,9 @@ void formatConnection(QTextStream &str, const SignalSlot &sender, const SignalSl
break; break;
case Language::Python: case Language::Python:
str << sender.name << '.' str << sender.name << '.'
<< sender.signature.leftRef(sender.signature.indexOf(QLatin1Char('('))) << QStringView{sender.signature}.left(sender.signature.indexOf(QLatin1Char('(')))
<< ".connect(" << receiver.name << '.' << ".connect(" << receiver.name << '.'
<< receiver.signature.leftRef(receiver.signature.indexOf(QLatin1Char('('))) << QStringView{receiver.signature}.left(receiver.signature.indexOf(QLatin1Char('(')))
<< ')'; << ')';
break; break;
} }

View File

@ -130,7 +130,7 @@ static inline bool isCppCommentChar(QChar c)
return c == QLatin1Char('/') || c == QLatin1Char('*'); return c == QLatin1Char('/') || c == QLatin1Char('*');
} }
static int leadingCppCommentCharCount(const QStringRef &s) static int leadingCppCommentCharCount(QStringView s)
{ {
int i = 0; int i = 0;
for (const int size = s.size(); i < size && isCppCommentChar(s.at(i)); ++i) { for (const int size = s.size(); i < size && isCppCommentChar(s.at(i)); ++i) {
@ -142,7 +142,7 @@ void Uic::writeCopyrightHeaderPython(const DomUI *ui) const
{ {
QString comment = ui->elementComment(); QString comment = ui->elementComment();
if (!comment.isEmpty()) { if (!comment.isEmpty()) {
const auto lines = comment.splitRef(QLatin1Char('\n')); const auto lines = QStringView{comment}.split(QLatin1Char('\n'));
for (const auto &line : lines) { for (const auto &line : lines) {
if (const int leadingCommentChars = leadingCppCommentCharCount(line)) { if (const int leadingCommentChars = leadingCppCommentCharCount(line)) {
out << language::repeat(leadingCommentChars, '#') out << language::repeat(leadingCommentChars, '#')