[PATCH] D26466: [clang-tidy] Fix NOLINT test

2017-01-24 Thread Alexander Kornienko via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL292926: [clang-tidy] Fix NOLINT test (authored by alexfh). Changed prior to commit: https://reviews.llvm.org/D26466?vs=77378&id=85571#toc Repository: rL LLVM https://reviews.llvm.org/D26466 Files:

[PATCH] D26466: [clang-tidy] Fix NOLINT test

2017-01-23 Thread Nikita Kakuev via Phabricator via cfe-commits
nkakuev added a comment. Ping. https://reviews.llvm.org/D26466 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26466: [clang-tidy] Fix NOLINT test

2016-11-09 Thread Nikita Kakuev via cfe-commits
nkakuev added a comment. @alexfh, you're welcome! Can you please commit this patch? https://reviews.llvm.org/D26466 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26466: [clang-tidy] Fix NOLINT test

2016-11-09 Thread Alexander Kornienko via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. Thanks! LG https://reviews.llvm.org/D26466 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo

[PATCH] D26466: [clang-tidy] Fix NOLINT test

2016-11-09 Thread Nikita Kakuev via cfe-commits
nkakuev created this revision. nkakuev added a reviewer: alexfh. nkakuev added a subscriber: cfe-commits. Test cases I've added in review https://reviews.llvm.org/D26218 were too brittle and weren't working properly. This patch fixes this. https://reviews.llvm.org/D26466 Files: test/clang-ti