diff options
| author | 2021-04-06 20:06:18 +0100 | |
|---|---|---|
| committer | 2021-04-07 10:36:11 +0100 | |
| commit | 942fd2bcfd384a12c900999fe663202c87319a68 (patch) | |
| tree | c2bad1906af27afbc3c7d96c3e5ca3c27c83f090 /src/modules/m_websocket.cpp | |
| parent | Merge branch 'insp3' into master. (diff) | |
Switch simple iterator loops to use range-based for loops.
Diffstat (limited to 'src/modules/m_websocket.cpp')
| -rw-r--r-- | src/modules/m_websocket.cpp | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/src/modules/m_websocket.cpp b/src/modules/m_websocket.cpp index 9bbf6ffa0..c31aa91b7 100644 --- a/src/modules/m_websocket.cpp +++ b/src/modules/m_websocket.cpp @@ -272,10 +272,10 @@ class WebSocketHook : public IOHookMiddle return result; // Strip out any CR+LF which may have been erroneously sent. - for (std::string::const_iterator iter = appdata.begin(); iter != appdata.end(); ++iter) + for (const auto& chr : appdata) { - if (*iter != '\r' && *iter != '\n') - destrecvq.push_back(*iter); + if (chr != '\r' && chr != '\n') + destrecvq.push_back(chr); } // If we are on the final message of this block append a line terminator. @@ -330,9 +330,9 @@ class WebSocketHook : public IOHookMiddle if (originheader.Find(recvq, "Origin:", 7, reqend)) { const std::string origin = originheader.ExtractValue(recvq); - for (WebSocketConfig::OriginList::const_iterator iter = config.allowedorigins.begin(); iter != config.allowedorigins.end(); ++iter) + for (const auto& cfgorigin : config.allowedorigins) { - if (InspIRCd::Match(origin, *iter, ascii_case_insensitive_map)) + if (InspIRCd::Match(origin, cfgorigin, ascii_case_insensitive_map)) { allowedorigin = true; break; @@ -363,9 +363,9 @@ class WebSocketHook : public IOHookMiddle // Nothing to do here. } - for (WebSocketConfig::ProxyRanges::const_iterator iter = config.proxyranges.begin(); iter != config.proxyranges.end(); ++iter) + for (const auto& proxyrange : config.proxyranges) { - if (InspIRCd::MatchCIDR(luser->GetIPString(), *iter, ascii_case_insensitive_map)) + if (InspIRCd::MatchCIDR(luser->GetIPString(), proxyrange, ascii_case_insensitive_map)) { // Give the user their real IP address. if (realsa != luser->client_sa) @@ -426,11 +426,11 @@ class WebSocketHook : public IOHookMiddle return (mysendq.empty() ? 0 : 1); std::string message; - for (StreamSocket::SendQueue::const_iterator elem = uppersendq.begin(); elem != uppersendq.end(); ++elem) + for (const auto& elem : uppersendq) { - for (StreamSocket::SendQueue::Element::const_iterator chr = elem->begin(); chr != elem->end(); ++chr) + for (const auto& chr : elem) { - if (*chr == '\n') + if (chr == '\n') { // We have found an entire message. Send it in its own frame. if (config.sendastext) @@ -450,9 +450,9 @@ class WebSocketHook : public IOHookMiddle } message.clear(); } - else if (*chr != '\r') + else if (chr != '\r') { - message.push_back(*chr); + message.push_back(chr); } } } |
