Mirrors_Framework/esphome
1
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2026-05-24 01:37:15 +08:00
Code Issues Actions 9 Packages Projects Releases Wiki Activity

[lock] Store state strings in flash and avoid heap allocation in set_state (#13729)
CI / Create common environment (push) Has been cancelled
Details
CI / Check pylint (push) Has been cancelled
Details
CI / Run script/ci-custom (push) Has been cancelled
Details
CI / Run pytest (macOS-latest, 3.11) (push) Has been cancelled
Details
CI / Run pytest (ubuntu-latest, 3.11) (push) Has been cancelled
Details
CI / Run pytest (ubuntu-latest, 3.13) (push) Has been cancelled
Details
CI / Run pytest (windows-latest, 3.11) (push) Has been cancelled
Details
CI / Determine which jobs to run (push) Has been cancelled
Details
CI / Run integration tests (push) Has been cancelled
Details
CI / Run C++ unit tests (push) Has been cancelled
Details
CI / Run script/clang-tidy for ESP32 IDF (push) Has been cancelled
Details
CI / Run script/clang-tidy for ESP8266 (push) Has been cancelled
Details
CI / Run script/clang-tidy for ZEPHYR (push) Has been cancelled
Details
CI / Run script/clang-tidy for ESP32 Arduino (push) Has been cancelled
Details
CI / Run script/clang-tidy for ESP32 Arduino 1/4 (push) Has been cancelled
Details
CI / Run script/clang-tidy for ESP32 Arduino 2/4 (push) Has been cancelled
Details
CI / Run script/clang-tidy for ESP32 Arduino 3/4 (push) Has been cancelled
Details
CI / Run script/clang-tidy for ESP32 Arduino 4/4 (push) Has been cancelled
Details
CI / Test components batch (${{ matrix.components }}) (push) Has been cancelled
Details
CI / pre-commit.ci lite (push) Has been cancelled
Details
CI / Build target branch for memory impact (push) Has been cancelled
Details
CI / Build PR branch for memory impact (push) Has been cancelled
Details
CI / Comment memory impact (push) Has been cancelled
Details
CI / CI Status (push) Has been cancelled
Details

Browse Source
This commit is contained in:
J. Nick Koston
2026-02-03 05:29:24 +01:00
committed by GitHub
parent ff6f7d3248
commit d4110bf650
2 changed files with 11 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
esphome/components/lock/lock.cpp
+9 -8
View File
@@ -2,6 +2,7 @@
#include "esphome/core/defines.h"
#include "esphome/core/controller_registry.h"
#include "esphome/core/log.h"
#include "esphome/core/progmem.h"
namespace esphome::lock {
@@ -84,21 +85,21 @@ LockCall &LockCall::set_state(optional<LockState> state) {
this->state_ = state;
return *this;
}
LockCall &LockCall::set_state(const std::string &state) {
if (str_equals_case_insensitive(state, "LOCKED")) {
LockCall &LockCall::set_state(const char *state) {
if (ESPHOME_strcasecmp_P(state, ESPHOME_PSTR("LOCKED")) == 0) {
this->set_state(LOCK_STATE_LOCKED);
} else if (str_equals_case_insensitive(state, "UNLOCKED")) {
} else if (ESPHOME_strcasecmp_P(state, ESPHOME_PSTR("UNLOCKED")) == 0) {
this->set_state(LOCK_STATE_UNLOCKED);
} else if (str_equals_case_insensitive(state, "JAMMED")) {
} else if (ESPHOME_strcasecmp_P(state, ESPHOME_PSTR("JAMMED")) == 0) {
this->set_state(LOCK_STATE_JAMMED);
} else if (str_equals_case_insensitive(state, "LOCKING")) {
} else if (ESPHOME_strcasecmp_P(state, ESPHOME_PSTR("LOCKING")) == 0) {
this->set_state(LOCK_STATE_LOCKING);
} else if (str_equals_case_insensitive(state, "UNLOCKING")) {
} else if (ESPHOME_strcasecmp_P(state, ESPHOME_PSTR("UNLOCKING")) == 0) {
this->set_state(LOCK_STATE_UNLOCKING);
} else if (str_equals_case_insensitive(state, "NONE")) {
} else if (ESPHOME_strcasecmp_P(state, ESPHOME_PSTR("NONE")) == 0) {
this->set_state(LOCK_STATE_NONE);
} else {
ESP_LOGW(TAG, "'%s' - Unrecognized state %s", this->parent_->get_name().c_str(), state.c_str());
ESP_LOGW(TAG, "'%s' - Unrecognized state %s", this->parent_->get_name().c_str(), state);
}
return *this;
}
esphome/components/lock/lock.h
+2 -1
View File
@@ -83,7 +83,8 @@ class LockCall {
/// Set the state of the lock device.
LockCall &set_state(optional<LockState> state);
/// Set the state of the lock device based on a string.
LockCall &set_state(const std::string &state);
LockCall &set_state(const char *state);
LockCall &set_state(const std::string &state) { return this->set_state(state.c_str()); }
void perform();