[PATCH] D67737: [clang-tidy] Add check for classes missing -hash ⚠️

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 18 17:31:43 PDT 2019


Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/objc/MissingHashCheck.cpp:40
+void MissingHashCheck::registerMatchers(MatchFinder *Finder) {
+  Finder->addMatcher(
+      objcMethodDecl(
----------------
Should check if language is Objective-C. See ForbiddenSubclassingCheck.cpp as example. Will be good ensure that all Objective-C checks do this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67737





More information about the cfe-commits mailing list