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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to