jeans/esphome
jeans/esphome
1
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2025-07-26 21:26:41 +00:00
Code Issues Packages Projects Releases Wiki Activity

address clang-tidy

Browse Source
This commit is contained in:
J. Nick Koston 2025-07-22 09:56:30 -10:00
parent ed61a2b77e
commit 2d15a3c0e8
No known key found for this signature in database
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
esphome/components/api/homeassistant_service.h
View File

@ -5,6 +5,7 @@
#include "api_pb2.h" #include "api_pb2.h"
#include "esphome/core/automation.h" #include "esphome/core/automation.h"
#include "esphome/core/helpers.h" #include "esphome/core/helpers.h"
#include <utility>
#include <vector> #include <vector>
namespace esphome { namespace esphome {
@ -36,10 +37,10 @@ template<typename... X> class TemplatableStringValue : public TemplatableValue<s
template<typename... Ts> class TemplatableKeyValuePair { template<typename... Ts> class TemplatableKeyValuePair {
public: public:
// It's safe to use const std::string& for the key parameter because keys are always // Keys are always string literals from YAML dictionary keys (e.g., "code", "event")
// string literals from YAML dictionary keys (e.g., "code", "event") and never // and never templatable values or lambdas. Only the value parameter can be a lambda/template.
// templatable values or lambdas. Only the value parameter can be a lambda/template. // Using pass-by-value with std::move allows optimal performance for both lvalues and rvalues.
template<typename T> TemplatableKeyValuePair(const std::string &key, T value) : key(key), value(value) {} template<typename T> TemplatableKeyValuePair(std::string key, T value) : key(std::move(key)), value(value) {}
std::string key; std::string key;
TemplatableStringValue<Ts...> value; TemplatableStringValue<Ts...> value;
}; };