This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Fix NOLINT test
ClosedPublic

Authored by nkakuev on Nov 9 2016, 10:58 AM.

Details

Summary

Test cases I've added in review D26218 were too brittle and weren't working properly.
This patch fixes this.

Diff Detail

Repository
rL LLVM

Event Timeline

nkakuev updated this revision to Diff 77378.Nov 9 2016, 10:58 AM
nkakuev retitled this revision from to [clang-tidy] Fix NOLINT test.
nkakuev updated this object.
nkakuev added a reviewer: alexfh.
nkakuev added a subscriber: cfe-commits.
alexfh accepted this revision.Nov 9 2016, 3:56 PM
alexfh edited edge metadata.

Thanks! LG

This revision is now accepted and ready to land.Nov 9 2016, 3:56 PM

@alexfh, you're welcome!
Can you please commit this patch?

This revision was automatically updated to reflect the committed changes.