[PATCH] D116425: [clang-tidy] Improve modernize-redundant-void-arg to recognize macro uses

Richard via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 11 11:32:59 PST 2022


LegalizeAdulthood added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/modernize/RedundantVoidArgCheck.cpp:157-161
+      if (ProtoToken.is(tok::TokenKind::l_paren)) {
+        State = TokenState::LeftParen;
+      } else if (isMacroIdentifier(Idents, ProtoToken)) {
+        State = TokenState::MacroId;
+      }
----------------
aaron.ballman wrote:
> 
Ah, found it in the style guide
https://llvm.org/docs/CodingStandards.html#don-t-use-braces-on-simple-single-statement-bodies-of-if-else-loop-statements


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116425/new/

https://reviews.llvm.org/D116425



More information about the cfe-commits mailing list