[PATCH] D57435: [clang-tidy] Add abseil-wrap-unique check

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 31 02:22:49 PST 2019


MyDeveloperDay added inline comments.


================
Comment at: clang-tidy/abseil/WrapUniqueCheck.h:27
+private:
+  std::string getArgs(const SourceManager *SM, const CallExpr *MemExpr);
+
----------------
Nit: could this method could be static in the cpp file and not in the header?


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D57435





More information about the cfe-commits mailing list