Re: [PATCH] D24845: [clang-tidy] fix for NOLINT after macro expansion

2016-09-24 Thread Matthias Gehre via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL282330: [clang-tidy] fix for NOLINT after macro expansion (authored by mgehre). Changed prior to commit: https://reviews.llvm.org/D24845?vs=72214=72391#toc Repository: rL LLVM

[PATCH] D24845: [clang-tidy] fix for NOLINT after macro expansion

2016-09-22 Thread Matthias Gehre via cfe-commits
mgehre created this revision. mgehre added reviewers: alexfh, aaron.ballman, hokein. mgehre added a subscriber: cfe-commits. When having ``` c++ #define MACRO code-with-warning MACRO; // NOLINT ``` clang-tidy would still show the warning, because it searched for "NOLINT" only in the first