[PATCH] D57184: [clang-format] Allow configuring list of function-like macros that resolve to a type

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 22 13:31:38 PDT 2019


Typz added a comment.
Herald added a project: clang.

Regarding variable/setting names, one issue I see is that these macros are not really type //names// IMO : the macro name is just part of the type (like a template), and its invocation is a type declaration. So maybe renaming to `Typedecl` or something like this would be slightly more representative.

Other than this, LGTM.



================
Comment at: docs/ClangFormatStyleOptions.rst:1237
+**TypenameMacros** (``std::vector<std::string>``)
+  A vector of macros that should be interpreted as types
+  instead of as function calls.
----------------
"type declarations" may be clearer?


Repository:
  rC Clang

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

https://reviews.llvm.org/D57184





More information about the cfe-commits mailing list