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

[MyDeveloperDay via Phabricator via cfe-commits]

MyDeveloperDay updated this revision to Diff 187155.
MyDeveloperDay marked 10 inline comments as done.
MyDeveloperDay added a comment.

Address review comments

- change OverrideMacro to OverrideSpelling
- change FinalMacro to FinalSpelling
- fix unit tests
- show warning without fix-it if Macros are not defined


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.187155.patch
Type: text/x-patch
Size: 12388 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190217/9fa933cf/attachment-0001.bin>