diff options
author | Sona Kurazyan <sona.kurazyan@qt.io> | 2022-04-26 14:46:11 +0200 |
---|---|---|
committer | Sona Kurazyan <sona.kurazyan@qt.io> | 2022-04-27 16:08:14 +0200 |
commit | c099ed5e3f28fb3cc69b48b1113c10938c6b1990 (patch) | |
tree | c0433e1c0b66f734b1efcaee92f01a764934bbbf /src/xml/dom | |
parent | e8562b0e8d6ffdb5b1d237d5f441ab1c9239b168 (diff) |
QtXml: stop using QLatin1Char constructor for creating char literals
Required for porting away from QLatin1Char/QLatin1String in scope of
QTBUG-98434.
Change-Id: Ic3e2391dc104f9f4f580166ce4211040fbda7bb0
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Diffstat (limited to 'src/xml/dom')
-rw-r--r-- | src/xml/dom/qdom.cpp | 44 | ||||
-rw-r--r-- | src/xml/dom/qdomhelpers.cpp | 4 |
2 files changed, 22 insertions, 26 deletions
diff --git a/src/xml/dom/qdom.cpp b/src/xml/dom/qdom.cpp index a36d4389e4..0047844eb0 100644 --- a/src/xml/dom/qdom.cpp +++ b/src/xml/dom/qdom.cpp @@ -105,7 +105,7 @@ QT_BEGIN_NAMESPACE */ static void qt_split_namespace(QString& prefix, QString& name, const QString& qName, bool hasURI) { - int i = qName.indexOf(QLatin1Char(':')); + qsizetype i = qName.indexOf(u':'); if (i == -1) { if (hasURI) prefix = QLatin1String(""); @@ -175,7 +175,7 @@ static QString fixedXmlName(const QString &_name, bool *ok, bool namespaces = fa *ok = true; if (namespaces && !prefix.isEmpty()) - return prefix + QLatin1Char(':') + result; + return prefix + u':' + result; return result; } @@ -309,13 +309,12 @@ static QString fixedPubidLiteral(const QString &data, bool *ok) return QString(); } - if (result.indexOf(QLatin1Char('\'')) != -1 - && result.indexOf(QLatin1Char('"')) != -1) { + if (result.indexOf(u'\'') != -1 && result.indexOf(u'"') != -1) { if (QDomImplementationPrivate::invalidDataPolicy == QDomImplementation::ReturnNullNode) { *ok = false; return QString(); } else { - result.remove(QLatin1Char('\'')); + result.remove(u'\''); } } @@ -335,13 +334,12 @@ static QString fixedSystemLiteral(const QString &data, bool *ok) QString result = data; - if (result.indexOf(QLatin1Char('\'')) != -1 - && result.indexOf(QLatin1Char('"')) != -1) { + if (result.indexOf(u'\'') != -1 && result.indexOf(u'"') != -1) { if (QDomImplementationPrivate::invalidDataPolicy == QDomImplementation::ReturnNullNode) { *ok = false; return QString(); } else { - result.remove(QLatin1Char('\'')); + result.remove(u'\''); } } @@ -1627,7 +1625,7 @@ QString QDomNode::nodeName() const return QString(); if (!IMPL->prefix.isEmpty()) - return IMPL->prefix + QLatin1Char(':') + IMPL->name; + return IMPL->prefix + u':' + IMPL->name; return IMPL->name; } @@ -3080,9 +3078,7 @@ QDomNodePrivate* QDomDocumentTypePrivate::appendChild(QDomNodePrivate* newChild) static QString quotedValue(const QString &data) { - QChar quote = data.indexOf(QLatin1Char('\'')) == -1 - ? QLatin1Char('\'') - : QLatin1Char('"'); + QChar quote = data.indexOf(u'\'') == -1 ? u'\'' : u'"'; return quote + data + quote; } @@ -3666,19 +3662,19 @@ static QString encodeText(const QString &str, while (i < len) { const QChar ati(retval.at(i)); - if (ati == QLatin1Char('<')) { + if (ati == u'<') { retval.replace(i, 1, QLatin1String("<")); len += 3; i += 4; - } else if (encodeQuotes && (ati == QLatin1Char('"'))) { + } else if (encodeQuotes && (ati == u'"')) { retval.replace(i, 1, QLatin1String(""")); len += 5; i += 6; - } else if (ati == QLatin1Char('&')) { + } else if (ati == u'&') { retval.replace(i, 1, QLatin1String("&")); len += 4; i += 5; - } else if (ati == QLatin1Char('>') && i >= 2 && retval[i - 1] == QLatin1Char(']') && retval[i - 2] == QLatin1Char(']')) { + } else if (ati == u'>' && i >= 2 && retval[i - 1] == u']' && retval[i - 2] == u']') { retval.replace(i, 1, QLatin1String(">")); len += 3; i += 4; @@ -3686,7 +3682,7 @@ static QString encodeText(const QString &str, (ati == QChar(0xA) || ati == QChar(0xD) || ati == QChar(0x9))) { - const QString replacement(QLatin1String("&#x") + QString::number(ati.unicode(), 16) + QLatin1Char(';')); + const QString replacement(QLatin1String("&#x") + QString::number(ati.unicode(), 16) + u';'); retval.replace(i, 1, replacement); i += replacement.length(); len += replacement.length() - 1; @@ -4049,7 +4045,7 @@ QString QDomElementPrivate::text() void QDomElementPrivate::save(QTextStream& s, int depth, int indent) const { if (!(prev && prev->isText())) - s << QString(indent < 1 ? 0 : depth * indent, QLatin1Char(' ')); + s << QString(indent < 1 ? 0 : depth * indent, u' '); QString qName(name); QString nsDecl(QLatin1String("")); @@ -4067,10 +4063,10 @@ void QDomElementPrivate::save(QTextStream& s, int depth, int indent) const if (prefix.isEmpty()) { nsDecl = QLatin1String(" xmlns"); } else { - qName = prefix + QLatin1Char(':') + name; + qName = prefix + u':' + name; nsDecl = QLatin1String(" xmlns:") + prefix; } - nsDecl += QLatin1String("=\"") + encodeText(namespaceURI) + QLatin1Char('\"'); + nsDecl += QLatin1String("=\"") + encodeText(namespaceURI) + u'\"'; } s << '<' << qName << nsDecl; @@ -4117,7 +4113,7 @@ void QDomElementPrivate::save(QTextStream& s, int depth, int indent) const s << Qt::endl; } QDomNodePrivate::save(s, depth + 1, indent); if (!last->isText()) - s << QString(indent < 1 ? 0 : depth * indent, QLatin1Char(' ')); + s << QString(indent < 1 ? 0 : depth * indent, u' '); s << "</" << qName << '>'; } else { @@ -4803,10 +4799,10 @@ void QDomCommentPrivate::save(QTextStream& s, int depth, int indent) const { /* We don't output whitespace if we would pollute a text node. */ if (!(prev && prev->isText())) - s << QString(indent < 1 ? 0 : depth * indent, QLatin1Char(' ')); + s << QString(indent < 1 ? 0 : depth * indent, u' '); s << "<!--" << value; - if (value.endsWith(QLatin1Char('-'))) + if (value.endsWith(u'-')) s << ' '; // Ensures that XML comment doesn't end with ---> s << "-->"; @@ -5211,7 +5207,7 @@ static QByteArray encodeEntity(const QByteArray& str) void QDomEntityPrivate::save(QTextStream& s, int, int) const { QString _name = name; - if (_name.startsWith(QLatin1Char('%'))) + if (_name.startsWith(u'%')) _name = QLatin1String("% ") + _name.mid(1); if (m_sys.isNull() && m_pub.isNull()) { diff --git a/src/xml/dom/qdomhelpers.cpp b/src/xml/dom/qdomhelpers.cpp index 459a2b5311..b8d25598c8 100644 --- a/src/xml/dom/qdomhelpers.cpp +++ b/src/xml/dom/qdomhelpers.cpp @@ -271,11 +271,11 @@ bool QDomParser::parseProlog() if (!reader->documentVersion().isEmpty()) { QString value(QLatin1String("version='")); value += reader->documentVersion(); - value += QLatin1Char('\''); + value += u'\''; if (!reader->documentEncoding().isEmpty()) { value += QLatin1String(" encoding='"); value += reader->documentEncoding(); - value += QLatin1Char('\''); + value += u'\''; } if (reader->isStandaloneDocument()) { value += QLatin1String(" standalone='yes'"); |