github/esphome
2
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2026-09-15 17:18:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

[core] Force inline Component::get_component_log_str()

Browse Source 
Move get_component_log_str() from component.cpp into the header
as an always_inline method. This is a trivial one-liner that just
calls component_source_lookup(), but the compiler was keeping it
out-of-line across 8 call sites (13 bytes).

Saves 8 bytes on ESP32.
This commit is contained in:
J. Nick Koston
2026-04-01 14:23:47 -10:00
parent 813b142b72
commit 5e7895bf26
2 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
esphome/core/component.cpp
-3
View File
@@ -267,9 +267,6 @@ void Component::call() {
break;
}
}
const LogString *Component::get_component_log_str() const {
return component_source_lookup(this->component_source_index_);
}
bool Component::should_warn_of_blocking(uint32_t blocking_time) {
// Convert centisecond threshold to milliseconds for comparison
uint32_t threshold_ms = static_cast<uint32_t>(this->warn_if_blocking_over_) * 10U;
esphome/core/component.h
+3 -1
View File
@@ -323,7 +323,9 @@ class Component {
*
* Returns LOG_STR("<unknown>") if source not set
*/
const LogString *get_component_log_str() const;
inline const LogString *get_component_log_str() const ESPHOME_ALWAYS_INLINE {
return component_source_lookup(this->component_source_index_);
}
bool should_warn_of_blocking(uint32_t blocking_time);