jeans/esphome
1
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2025-08-10 20:29:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

[core] Replace std::find and std::max_element with simple loops to reduce binary size

Browse Source
This commit is contained in:
J. Nick Koston
2025-08-02 21:46:56 -10:00
parent 296442d8f1
commit 451095eef4
1 changed files with 11 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
esphome/core/application.cpp
View File

@@ -459,24 +459,24 @@ void Application::unregister_socket_fd(int fd) {
if (fd < 0)
return;
auto it = std::find(this->socket_fds_.begin(), this->socket_fds_.end(), fd);
if (it != this->socket_fds_.end()) {
for (size_t i = 0; i < this->socket_fds_.size(); i++) {
if (this->socket_fds_[i] != fd)
continue;
// Swap with last element and pop - O(1) removal since order doesn't matter
if (it != this->socket_fds_.end() - 1) {
std::swap(*it, this->socket_fds_.back());
}
if (i < this->socket_fds_.size() - 1)
this->socket_fds_[i] = this->socket_fds_.back();
this->socket_fds_.pop_back();
this->socket_fds_changed_ = true;
// Only recalculate max_fd if we removed the current max
if (fd == this->max_fd_) {
if (this->socket_fds_.empty()) {
this->max_fd_ = -1;
} else {
// Find new max using std::max_element
this->max_fd_ = *std::max_element(this->socket_fds_.begin(), this->socket_fds_.end());
}
this->max_fd_ = -1;
for (int sock_fd : this->socket_fds_)
if (sock_fd > this->max_fd_)
this->max_fd_ = sock_fd;
}
return;
}
}