kwk 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. + ---------------- Eugene.Zelenko wrote: > 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. Agreed, that makes sense. Sorry for not getting that right away. Afterall I did write about that above :) > Yes modernize-use-equals-delete does, this but it only notifies about it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80531/new/ https://reviews.llvm.org/D80531 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits