[PATCH] D68074: [clang-tidy] Add readability-make-member-function-const

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 1 20:49:29 PDT 2019


gribozavr2 added inline comments.


================
Comment at: clang-tools-extra/test/clang-tidy/readability-make-member-function-const.cpp:321
+
+} // namespace Keep
----------------
Could you add tests for calls to members of `this` through member function pointers? I don't care about possible false negatives in that case, but the code shouldn't crash or produce false positives.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68074





More information about the cfe-commits mailing list