[PATCH] D66365: [clang-tidy] [readability-convert-member-functions-to-static] ignore functions that hide base class methods

Matthias Gehre via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 19 12:25:41 PDT 2019


mgehre added a comment.

I struggle myself to fully understand the motivation behind the bug report. This is certainly not code that I would write.
So I don't really care about that kind of code and I didn't want to be in the way of other people's way of writing C++.

Alternatively, we could ask the bug reporter to use `// NOLINT` to support his specific case. I will try to get some clarification
from the reporter.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66365





More information about the cfe-commits mailing list