diff options
-rw-r--r-- | src/network/access/qnetworkcookiejar.cpp | 2 | ||||
-rw-r--r-- | src/network/kernel/qtldurl.cpp | 6 | ||||
-rw-r--r-- | src/network/kernel/qtldurl_p.h | 4 |
3 files changed, 7 insertions, 5 deletions
diff --git a/src/network/access/qnetworkcookiejar.cpp b/src/network/access/qnetworkcookiejar.cpp index af5b126953..0856dd2252 100644 --- a/src/network/access/qnetworkcookiejar.cpp +++ b/src/network/access/qnetworkcookiejar.cpp @@ -47,7 +47,7 @@ #include "private/qtldurl_p.h" #else QT_BEGIN_NAMESPACE -static bool qIsEffectiveTLD(QString domain) +static bool qIsEffectiveTLD(QStringView domain) { // provide minimal checking by not accepting cookies on real TLDs return !domain.contains(QLatin1Char('.')); diff --git a/src/network/kernel/qtldurl.cpp b/src/network/kernel/qtldurl.cpp index c2f7d1de26..d93407d6f5 100644 --- a/src/network/kernel/qtldurl.cpp +++ b/src/network/kernel/qtldurl.cpp @@ -114,9 +114,11 @@ Q_NETWORK_EXPORT QString qTopLevelDomain(const QString &domain) \internal Return true if \a domain is a top-level-domain per Qt's copy of the Mozilla public suffix list. + + The \a domain must be in lower-case format (as per QString::toLower()). */ -Q_NETWORK_EXPORT bool qIsEffectiveTLD(const QStringRef &domain) +Q_NETWORK_EXPORT bool qIsEffectiveTLD(QStringView domain) { // for domain 'foo.bar.com': // 1. return if TLD table contains 'foo.bar.com' @@ -126,7 +128,7 @@ Q_NETWORK_EXPORT bool qIsEffectiveTLD(const QStringRef &domain) if (containsTLDEntry(domain, ExactMatch)) // 1 return true; - const int dot = domain.indexOf(QLatin1Char('.')); + const auto dot = domain.indexOf(QLatin1Char('.')); if (dot < 0) // Actual TLD: may be effective if the subject of a wildcard rule: return containsTLDEntry(QString(QLatin1Char('.') + domain), SuffixMatch); if (containsTLDEntry(domain.mid(dot), SuffixMatch)) // 2 diff --git a/src/network/kernel/qtldurl_p.h b/src/network/kernel/qtldurl_p.h index a6948cfd48..c0a07d8e00 100644 --- a/src/network/kernel/qtldurl_p.h +++ b/src/network/kernel/qtldurl_p.h @@ -60,10 +60,10 @@ QT_REQUIRE_CONFIG(topleveldomain); QT_BEGIN_NAMESPACE Q_NETWORK_EXPORT QString qTopLevelDomain(const QString &domain); -Q_NETWORK_EXPORT bool qIsEffectiveTLD(const QStringRef &domain); +Q_NETWORK_EXPORT bool qIsEffectiveTLD(QStringView domain); inline bool qIsEffectiveTLD(const QString &domain) { - return qIsEffectiveTLD(QStringRef(&domain)); + return qIsEffectiveTLD(qToStringViewIgnoringNull(domain)); } QT_END_NAMESPACE |