[PATCH] D90392: [clang-tidy] Omit std::make_unique/make_shared for default initialization.

Mon Nov 9 22:17:05 PST 2020

lebedev.ri added a comment.

In D90392#2362118 <https://reviews.llvm.org/D90392#2362118>, @njames93 wrote:

> IIUC, this is handling the case where `Ptr.reset(new int)` which is different to `Ptr.reset(new int())` because the former doesn't initialise the int while the latter default(zero) initialises it.
> If that's correct I still think we should still warn about this case, but don't suggest an auto-fix as that will change behaviour.
> Maybe put a note explaining why it can't be auto-fixed.

+1

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90392