[PATCH] D57087: [clang-tidy] add OverrideMacro to modernize-use-override check
Jonas Toth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 28 10:26:22 PST 2019
JonasToth accepted this revision.
JonasToth added a comment.
LGTM
================
Comment at: clang-tidy/modernize/UseOverrideCheck.cpp:196
+ ReplacementText = " " + OverrideSpelling + " ";
+ }
} else if (GetText(Tokens.back(), Sources) == "ABSTRACT") {
----------------
Nit: braces
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57087/new/
https://reviews.llvm.org/D57087
More information about the cfe-commits
mailing list