[PATCH] D70265: [clang-tidy] Add clang tidy check I.2 to cppcoreguidelines
Kim Viggedal via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 30 07:07:16 PST 2019
vingeldal marked an inline comment as done.
vingeldal added inline comments.
================
Comment at: clang-tools-extra/clang-tidy/cppcoreguidelines/AvoidNonConstGlobalVariablesCheck.cpp:110
+ "making it const")
+ << MemberVariable; // FIXME: Add fix-it hint to MemberVariable
+ }
----------------
JonasToth wrote:
> missing `return`?
No, one of the below matchers may also be valid, in case we have a non-const member variable which is also a pointer or reference to non-const data.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70265/new/
https://reviews.llvm.org/D70265
More information about the cfe-commits
mailing list