Commit b05ec56d authored by André Klitzing's avatar André Klitzing
Browse files

Replace iterator based loop by range-for-loop


Found by clazy and fixes "clazy-strict-iterators" warning.

Change-Id: I8f50c92ac054d627499440d593faa43ed5b4ad4b
Reviewed-by: default avatarMårten Nordheim <marten.nordheim@qt.io>
Reviewed-by: default avatarEdward Welbourne <edward.welbourne@qt.io>
Showing with 9 additions and 9 deletions
...@@ -318,17 +318,17 @@ void QWebSocketHandshakeRequest::readHandshake(QTextStream &textStream, int maxH ...@@ -318,17 +318,17 @@ void QWebSocketHandshakeRequest::readHandshake(QTextStream &textStream, int maxH
//optional headers //optional headers
m_origin = m_headers.value(QStringLiteral("origin"), QString()); m_origin = m_headers.value(QStringLiteral("origin"), QString());
const QStringList protocolLines = m_headers.values(QStringLiteral("sec-websocket-protocol")); const QStringList protocolLines = m_headers.values(QStringLiteral("sec-websocket-protocol"));
for (QStringList::const_iterator pl = protocolLines.begin(); pl != protocolLines.end(); ++pl) { for (const QString& pl : protocolLines) {
QStringList protocols = (*pl).split(QStringLiteral(","), QString::SkipEmptyParts); const QStringList protocols = pl.split(QStringLiteral(","), QString::SkipEmptyParts);
for (QStringList::const_iterator p = protocols.begin(); p != protocols.end(); ++p) for (const QString& p : protocols)
m_protocols << (*p).trimmed(); m_protocols << p.trimmed();
} }
const QStringList extensionLines = m_headers.values(QStringLiteral("sec-websocket-extensions")); const QStringList extensionLines = m_headers.values(QStringLiteral("sec-websocket-extensions"));
for (QStringList::const_iterator el = extensionLines.begin(); for (const QString& el : extensionLines) {
el != extensionLines.end(); ++el) { const QStringList extensions = el.split(QStringLiteral(","), QString::SkipEmptyParts);
QStringList extensions = (*el).split(QStringLiteral(","), QString::SkipEmptyParts); for (const QString& e : extensions)
for (QStringList::const_iterator e = extensions.begin(); e != extensions.end(); ++e) m_extensions << e.trimmed();
m_extensions << (*e).trimmed();
} }
//TODO: authentication field //TODO: authentication field
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment