jeans/esphome
jeans/esphome
1
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2025-07-27 05:36:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add missing clang-tidy NOLINT comments for ArduinoJson v7 in IDF webserver (#9508)

Browse Source
This commit is contained in:
J. Nick Koston 2025-07-14 20:26:54 -10:00 committed by GitHub
parent 63b8a219e6
commit b959baf3d6
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
esphome/components/web_server_idf/web_server_idf.cpp
View File

@ -389,10 +389,12 @@ AsyncEventSourceResponse::AsyncEventSourceResponse(const AsyncWebServerRequest *
#ifdef USE_WEBSERVER_SORTING #ifdef USE_WEBSERVER_SORTING
for (auto &group : ws->sorting_groups_) { for (auto &group : ws->sorting_groups_) {
// NOLINTBEGIN(clang-analyzer-cplusplus.NewDeleteLeaks) false positive with ArduinoJson
message = json::build_json([group](JsonObject root) { message = json::build_json([group](JsonObject root) {
root["name"] = group.second.name; root["name"] = group.second.name;
root["sorting_weight"] = group.second.weight; root["sorting_weight"] = group.second.weight;
}); });
// NOLINTEND(clang-analyzer-cplusplus.NewDeleteLeaks)
// a (very) large number of these should be able to be queued initially without defer // a (very) large number of these should be able to be queued initially without defer
// since the only thing in the send buffer at this point is the initial ping/config // since the only thing in the send buffer at this point is the initial ping/config