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

Jonas Toth via Phabricator via llvm-commits llvm-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 llvm-commits mailing list