[PATCH] D55044: [clang-tidy] check for Abseil make_unique

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 13 05:06:54 PST 2018


hokein added a comment.

In fact, the existing `modernize-make-unique` can be configured to support `absl::make_unique`, you'd just need to configure the check option `MakeSmartPtrFunction` to `absl::make_unique` (this is what we do inside google).

The biggest missing part of the `modernize-make-unique` is `absl::WrapUnique`, I think we should extend `MakeSmartPtrCheck` class (maybe add hooks) to support it.


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

https://reviews.llvm.org/D55044





More information about the cfe-commits mailing list