mirror/esphome
1
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2026-02-20 08:25:35 -07:00
Code Issues Packages Projects Releases Wiki Activity

Add nolint for non-const buildinfo variables

Browse Source 
Variables must remain non-const to prevent compiler optimization
that would bypass the indirection workaround for PC-relative
relocation issues.
This commit is contained in:
David Woodhouse
2025-12-11 22:48:02 +09:00
parent 478f12f75e
commit 0b1ea8f2ca
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
esphome/core/buildinfo.cpp
View File

@@ -23,8 +23,10 @@ namespace buildinfo {
// Reference the linker symbols as uintptr_t from the *data* section to
// avoid issues with pc-relative relocations on 64-bit platforms.
// NOLINTBEGIN(cppcoreguidelines-avoid-non-const-global-variables)
static uintptr_t config_hash = (uintptr_t) &ESPHOME_CONFIG_HASH;
static uintptr_t build_time = (uintptr_t) &ESPHOME_BUILD_TIME;
// NOLINTEND(cppcoreguidelines-avoid-non-const-global-variables)
const char *get_config_hash() {
static char hash_str[9];