[libcxx-commits] [PATCH] D128146: [libc++] Use uninitialized algorithms for vector

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik added a comment.

@thakis I wouldn't say the change is intentional, but the change is definitely expected. Especially, since it looks like `-Wdeprecated-copy` is basically a rule-of-{0, 3, 5} warning, which you violated by having the copy constructor explicitly defaulted. Although I'm not sure why clang only warns when the constructor is used instead of when the class is declared. If you want you can use clang-tidy with `cppcoreguidelines-special-member-functions` to ensure that all your classes follow the rule-of-0-or-5.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128146