Merge pull request esphome#30 from ESP32Async/issues/ws-cleanup

fix(ws): break loop after erasing client

Author: mathieucarbou

src/AsyncWebSocket.cpp

@@ -916,11 +916,10 @@ void AsyncWebSocket::cleanupClients(uint16_t maxClients) {
     _clients.front().close();
   }
 
-  for (auto iter = std::begin(_clients); iter != std::end(_clients);) {
-    if (iter->shouldBeDeleted()) {
-      iter = _clients.erase(iter);
-    } else {
-      iter++;
+  for (auto i = _clients.begin(); i != _clients.end(); ++i) {
+    if (i->shouldBeDeleted()) {
+      _clients.erase(i);
+      break;
     }
   }
 }