[PATCH] D70165: clang-tidy: modernize-use-override new option AllowOverrideAndFinal

Conrad Poelman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 13 16:50:43 PST 2019


poelmanc added a comment.

In D70165#1744007 <https://reviews.llvm.org/D70165#1744007>, @JonasToth wrote:

> LGTM!
>  Did you check on a real code-base that suffers from the issue, if that works as expected?


Thanks! I have now run it on our real code base and it worked as expected.

I lack commit access and would appreciate anyone committing this on my behalf.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D70165





More information about the cfe-commits mailing list