[PATCH] D60507: [clang-tidy] new check: bugprone-unhandled-self-assignment

Sun Apr 14 06:34:20 PDT 2019

xazax.hun added inline comments.

================
Comment at: clang-tools-extra/clang-tidy/bugprone/UnhandledSelfAssignmentCheck.cpp:51
+  // Matcher for standard smart pointers.
+  const auto SmartPointerType = qualType(hasUnqualifiedDesugaredType(
+      recordType(hasDeclaration(classTemplateSpecializationDecl(
----------------
JonasToth wrote:
> what about `auto_ptr`? I am actually not sure if we should care, as its deprecated and removed already, on the other hand legacy code probably still has it.
I am perfectly fine with addressing this in a follow-up patch or even not addressing at all, but I think for some users it might be useful to be able to specify a set of suspicious types as a configuration option such as custom smart pointers, handles and so on. 

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60507