sascha/esphome
1
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2025-10-31 15:12:06 +00:00
Code Issues Releases Wiki Activity

make sure components that disable in setup are disabled at start

Browse Source
This commit is contained in:
J. Nick Koston
2025-06-17 23:40:31 +02:00
parent 4c37c20d76
commit 969abc3f29
1 changed files with 17 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
esphome/core/application.cpp
View File

@@ -244,11 +244,25 @@ void Application::teardown_components(uint32_t timeout_ms) {
void Application::calculate_looping_components_() { void Application::calculate_looping_components_() {
for (auto *obj : this->components_) { for (auto *obj : this->components_) {
if (obj->has_overridden_loop()) if (obj->has_overridden_loop()) {
this->looping_components_.push_back(obj); this->looping_components_.push_back(obj);
}
}
// Partition components based on their current state
// Components that have already called disable_loop() during setup (state == LOOP_DONE)
// should start in the inactive section of the partition
this->looping_components_active_end_ = 0;
for (uint16_t i = 0; i < this->looping_components_.size(); i++) {
Component *comp = this->looping_components_[i];
if ((comp->get_component_state() & COMPONENT_STATE_MASK) != COMPONENT_STATE_LOOP_DONE) {
// Component is active - swap it to the active section if needed
if (i != this->looping_components_active_end_) {
std::swap(this->looping_components_[i], this->looping_components_[this->looping_components_active_end_]);
}
this->looping_components_active_end_++;
}
} }
// Initially all components are active
this->looping_components_active_end_ = this->looping_components_.size();
} }
void Application::disable_component_loop_(Component *component) { void Application::disable_component_loop_(Component *component) {