This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] bug fix: Don't warn on partial template specialization in `misc-definitions-in-headers` check.
ClosedPublic

Authored by hokein on Jan 26 2016, 5:45 AM.

Diff Detail

Event Timeline

hokein updated this revision to Diff 45984.Jan 26 2016, 5:45 AM
hokein retitled this revision from to [clang-tidy] bug fix: Don't warn on partial template specialization in `misc-definitions-in-headers` check..
hokein updated this object.
hokein set the repository for this revision to rL LLVM.
hokein added a subscriber: cfe-commits.
alexfh accepted this revision.Jan 26 2016, 8:32 AM
alexfh edited edge metadata.

LG. Do you need me to commit the patch?

This revision is now accepted and ready to land.Jan 26 2016, 8:32 AM

Yes, please help to land the patch for me :). Thanks.

This revision was automatically updated to reflect the committed changes.
hokein added a comment.Feb 3 2016, 4:21 AM

@alexfh, I get commit access, and have committed the patch by myself (Seems it is a successful commit :D ).