[PATCH] D80531: [clang-tidy]: Added modernize-replace-disallow-copy-and-assign-macro
Eugene Zelenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue May 26 09:47:01 PDT 2020
Eugene.Zelenko added inline comments.
================
Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize-replace-disallow-copy-and-assign-macro.rst:31
+* Notice that the migration example above leaves the ``private`` access
+ specification untouched. This opens room for improvement, yes I know.
+
----------------
kwk wrote:
> Eugene.Zelenko wrote:
> > Please refer to //modernize-use-equals-delete// instead of second statement.
> `modernize-use-equals-delete` doesn't help because it also does not remove the `private` access. Why refer to it?
It complaines about deleted stuff in private section.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80531/new/
https://reviews.llvm.org/D80531
More information about the cfe-commits
mailing list