mirror/esphome
1
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2026-02-18 23:45:40 -07:00
Code Issues Packages Projects Releases Wiki Activity

Use if/else instead of continue in client loop

Browse Source
This commit is contained in:
J. Nick Koston
2026-02-09 20:42:25 -06:00
parent 0c62781539
commit a671f6ea85
1 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
esphome/components/api/api_server.cpp
View File

@@ -148,16 +148,14 @@ void APIServer::loop() {
while (client_index < this->clients_.size()) {
auto &client = this->clients_[client_index];
if (!client->flags_.remove) {
if (client->flags_.remove) {
// Rare case: handle disconnection (don't increment - swapped element needs processing)
this->remove_client_(client_index);
} else {
// Common case: process active client
client->loop();
client_index++;
continue;
}
// Rare case: handle disconnection
this->remove_client_(client_index);
// Don't increment client_index since we need to process the swapped element
}
}