[clang-tools-extra] [clang-tidy] Add modernize-make-direct check (PR #118120)

Oliver Stöneberg via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 29 11:38:44 PST 2024


firewave wrote:

This will conflict with `modernize-make-shared` and `modernize-make-unique`.

I also very sure having `new` any modern C++ code is very much to be avoided.

Having no insight on the differences of the inner workings - but just based on https://en.cppreference.com/w/cpp/memory/shared_ptr/make_shared#Notes this appears to be something behaving very differently and less optimal/safe.

https://github.com/llvm/llvm-project/pull/118120


More information about the cfe-commits mailing list