[PATCH] D57087: [clang-tidy] add OverrideMacro to modernize-use-override check

MyDeveloperDay via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 28 10:12:06 PST 2019


MyDeveloperDay updated this revision to Diff 188752.
MyDeveloperDay added a comment.

rebase after D58731: [clang-tidy] added cppcoreguidelines-explicit-virtual-functions <https://reviews.llvm.org/D58731> change

as this was previous accepted I will land shortly, but I'd appreciate a quick overview... @JonasToth, @lewmpk


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57087/new/

https://reviews.llvm.org/D57087

Files:
  clang-tidy/modernize/UseOverrideCheck.cpp
  clang-tidy/modernize/UseOverrideCheck.h
  docs/ReleaseNotes.rst
  docs/clang-tidy/checks/modernize-use-override.rst
  test/clang-tidy/modernize-use-override-with-macro.cpp
  test/clang-tidy/modernize-use-override-with-no-macro-inscope.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57087.188752.patch
Type: text/x-patch
Size: 13087 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190228/3e20a511/attachment.bin>


More information about the llvm-commits mailing list