diff options
| author | Matthias Gehre <M.Gehre@gmx.de> | 2016-09-24 16:06:53 +0000 |
|---|---|---|
| committer | Matthias Gehre <M.Gehre@gmx.de> | 2016-09-24 16:06:53 +0000 |
| commit | 41a83a7d2b654de40bc623821ff111e56191236c (patch) | |
| tree | 98cdb1bc879f0d4d8438b7b1853d6d673179ed5e /libcxx/test/std/diagnostics/syserr | |
| parent | 907cde3cc2e96b8558960641d2d2c78870a019b4 (diff) | |
| download | bcm5719-llvm-41a83a7d2b654de40bc623821ff111e56191236c.tar.gz bcm5719-llvm-41a83a7d2b654de40bc623821ff111e56191236c.zip | |
[clang-tidy] fix for NOLINT after macro expansion
Summary:
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 line,
not on the second.
This caused e.g. https://llvm.org/bugs/show_bug.cgi?id=29089
(where the macro was defined in a system header). See also
the added test cases.
Now clang-tidy looks at the line of macro invocation and every line
of macro definition for a NOLINT comment.
Reviewers: alexfh, aaron.ballman, hokein
Subscribers: cfe-commits
Differential Revision: https://reviews.llvm.org/D24845
llvm-svn: 282330
Diffstat (limited to 'libcxx/test/std/diagnostics/syserr')
0 files changed, 0 insertions, 0 deletions
