Skip to content

Commit 8ec38a0

Browse files
tjkl1mchehab
authored andcommitted
Strings: Replace QStringRef with QStringView
Signed-off-by: Tobias Klausmann <[email protected]>
1 parent dddc53f commit 8ec38a0

File tree

5 files changed

+23
-21
lines changed

5 files changed

+23
-21
lines changed

src/dvb/xmltv.cpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -85,10 +85,10 @@ DvbEpgLangEntry *XmlTv::getLangEntry(DvbEpgEntry &epgEntry,
8585
bool XmlTv::parseChannel(void)
8686
{
8787
const QString emptyString("");
88-
QStringRef empty(&emptyString);
88+
QStringView empty(emptyString);
8989

9090
const QXmlStreamAttributes attrs = r->attributes();
91-
QStringRef channelName = attrs.value("id");
91+
QStringView channelName = attrs.value("id");
9292
QList<QString>list;
9393

9494
QString current = r->name().toString();
@@ -103,7 +103,7 @@ bool XmlTv::parseChannel(void)
103103
if (t != QXmlStreamReader::StartElement)
104104
continue;
105105

106-
QStringRef name = r->name();
106+
QStringView name = r->name();
107107
if (name == QLatin1String("display-name")) {
108108
QString display = r->readElementText();
109109
list.append(display);
@@ -250,10 +250,10 @@ QString XmlTv::parseCredits(void)
250250
bool XmlTv::parseProgram(void)
251251
{
252252
const QString emptyString("");
253-
QStringRef empty(&emptyString);
253+
QStringView empty(emptyString);
254254

255255
QXmlStreamAttributes attrs = r->attributes();
256-
QStringRef channelName = attrs.value("channel");
256+
QStringView channelName = attrs.value("channel");
257257
QHash<QString, QList<QString>>::ConstIterator it;
258258

259259
it = channelMap.constFind(channelName.toString());
@@ -334,7 +334,7 @@ bool XmlTv::parseProgram(void)
334334
continue;
335335

336336
QString lang;
337-
QStringRef element = r->name();
337+
QStringView element = r->name();
338338
if (element == QLatin1String("title")) {
339339
attrs = r->attributes();
340340
lang = IsoCodes::code2Convert(attrs.value("lang").toString());
@@ -598,7 +598,7 @@ bool XmlTv::load(QString file)
598598
if (r->readNext() != QXmlStreamReader::StartElement)
599599
continue;
600600

601-
QStringRef name = r->name();
601+
QStringView name = r->name();
602602

603603
if (name == QLatin1String("channel")) {
604604
if (!parseChannel())

src/iso-codes.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ namespace IsoCodes
6161
bool inDoc = false;
6262
while (!r.atEnd()) {
6363
const QXmlStreamReader::TokenType t = r.readNext();
64-
QStringRef name;
64+
QStringView name;
6565
switch (t) {
6666
case QXmlStreamReader::StartElement:
6767
name = r.name();

tools/updatedvbsi.cpp

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,12 @@
2323
# define qInfo qDebug
2424
#endif
2525

26+
#include <QRegularExpression>
27+
#include <QString>
28+
2629
#include <QDomDocument>
2730
#include <QFile>
28-
#include <QRegularExpression>
31+
2932

3033
class Element
3134
{
@@ -206,7 +209,7 @@ void SiXmlParser::parseEntry(QDomNode node, Type type, QTextStream &headerStream
206209

207210
QString entryName = node.nodeName();
208211
QRegularExpression regFuncName("^Dvb|^Atsc");
209-
QString initFunctionName = QString(entryName).replace(regFuncName), "init");
212+
QString initFunctionName = QString(entryName).replace(regFuncName, "init");
210213
bool ignoreFirstNewLine = false;
211214

212215
QString funcName = entryName + "::";
@@ -257,8 +260,8 @@ void SiXmlParser::parseEntry(QDomNode node, Type type, QTextStream &headerStream
257260

258261
while (true) {
259262
int oldSize = entryLengthCalculation.size();
260-
QRegularExpression entryLengthRegEx("at\\(([0-9]*)\\)"), "quint8(data[\\1])");
261-
entryLengthCalculation.replace(entryLengthRegEx);
263+
QRegularExpression entryLengthRegEx("at\\(([0-9]*)\\)");
264+
entryLengthCalculation.replace(entryLengthRegEx, "quint8(data[\\1])");
262265

263266
if (entryLengthCalculation.size() == oldSize) {
264267
break;

tools/updatemimetypes.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ int main(int argc, char *argv[])
166166
}
167167
}
168168

169-
QRegularExpression regExp("\\*\\.[a-z0-9+]+");
169+
QRegularExpression regExp(QRegularExpression::anchoredPattern("\\*\\.[a-z0-9+]+"));
170170

171171
for (int i = 0; i < extensions.size(); ++i) {
172172
if (extensions.at(i) == "*.anim[1-9j]") {
@@ -187,7 +187,7 @@ int main(int argc, char *argv[])
187187
extensions.insert(i, "*.vdr");
188188
}
189189

190-
if (!regExp.exactMatch(extensions.at(i))) {
190+
if (!regExp.match(extensions.at(i)).hasMatch()) {
191191
qCritical() << "unknown extension syntax" << extensions.at(i);
192192
dontUpdate = true;
193193
}

tools/updatesource.cpp

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -128,9 +128,9 @@ int main(int argc, char *argv[])
128128
"is longer than 99 characters";
129129
}
130130

131-
int logIndex = logRegExp.indexIn(line);
132-
133-
if (logIndex >= 0) {
131+
QRegularExpressionMatch match = logRegExp.match(line);
132+
if (match.hasMatch()) {
133+
int logIndex = match.capturedStart();
134134
logIndex = (line.indexOf('"', logIndex) + 1);
135135

136136
if (!line.mid(logIndex).startsWith(logFunctionName)) {
@@ -153,11 +153,10 @@ int main(int argc, char *argv[])
153153

154154
if ((bracketLevel == 0) && !line.startsWith('\t')) {
155155
QRegularExpression logFunctionRegExp("[0-9A-Za-z:~]*[(]");
156-
int index = logFunctionRegExp.indexIn(line);
157-
158-
if (index >= 0) {
156+
QRegularExpressionMatch match = logFunctionRegExp.match(line);
157+
if (match.hasMatch()) {
159158
logFunctionName =
160-
line.mid(index, logFunctionRegExp.matchedLength());
159+
line.mid(match.capturedStart(), match.capturedLength());
161160
logFunctionName.chop(1);
162161
logFunctionName.append(": ");
163162
}

0 commit comments

Comments
 (0)