AlexanderLanin marked 5 inline comments as done.
AlexanderLanin added inline comments.
================
Comment at: clang-tidy/modernize/UseConstInsteadOfDefineCheck.cpp:86
+ else if (isAnyParenthesis(Tok) || isAnyNumericLiteral(Tok) ||
+ (!hasPrefix && isAnyCharLiteral(Tok))) {
+ // prefix shall not be accepted anymore after any number
----------------
aaron.ballman wrote:
> What about string literals? e.g., `#define NAME "foobar"`
Excluded (for now) due to string concatenation done by preprocessor when you
write NAME1 NAME2 in your code. I've added a section into
modernize-use-const-instead-of-define.rst to explain this.
https://reviews.llvm.org/D29692
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
