From eb9b4e44ee130c821317d513ccaf6ff51d492b7a Mon Sep 17 00:00:00 2001 From: Chris Hennes Date: Sun, 31 Aug 2025 20:48:12 -0500 Subject: [PATCH] Update clang-tidy checks to remove avoid-do-while While the advice is good, the overwhelming majority of "do-while" warnings that come up in our codebase are macros using a standard pattern using a do-while loop to encapsulate the macro's work. --- .clang-tidy | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.clang-tidy b/.clang-tidy index b76cba3e82..41a4460324 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -3,7 +3,7 @@ Checks: 'clang-diagnostic-*,clang-analyzer-*,boost-*,bugprone-*, performance-*,readability-*,portability-*,modernize-*,cppcoreguidelines-*,google-explicit-constructor, concurrency-*,-modernize-use-trailing-return-type, -modernize-use-nodiscard,-readability-identifier-length, -readability-redundant-access-specifiers,-readability-qualified-auto, --cppcoreguidelines-avoid-non-const-global-variables,-cppcoreguidelines-owning-memory, +-cppcoreguidelines-avoid-non-const-global-variables,-cppcoreguidelines-owning-memory,-cppcoreguidelines-avoid-do-while, -readability-convert-member-functions-to-static,-bugprone-easily-swappable-parameters, -cppcoreguidelines-pro-type-static-cast-downcast' WarningsAsErrors: ''