From db831ebee07dfbff2b48c5cf0a518380545b84d7 Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Wed, 11 Feb 2026 17:37:09 -0600 Subject: [PATCH] Fix clang-tidy: use const auto& for arg() return value On Arduino, arg() returns const String&, so auto copies unnecessarily. const auto& binds to the reference on Arduino and extends the temporary lifetime on IDF. --- esphome/components/web_server/web_server.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/esphome/components/web_server/web_server.h b/esphome/components/web_server/web_server.h index 081bf75aa7..c5e4c04f6f 100644 --- a/esphome/components/web_server/web_server.h +++ b/esphome/components/web_server/web_server.h @@ -574,7 +574,7 @@ class WebServer : public Controller, template void parse_bool_param_(AsyncWebServerRequest *request, ParamNameType param_name, T &call, Ret (T::*setter)(bool)) { if (request->hasArg(param_name)) { - auto param_value = request->arg(param_name); + const auto ¶m_value = request->arg(param_name); // First check on/off (default), then true/false (custom) auto val = parse_on_off(param_value.c_str()); if (val == PARSE_NONE) {