[PATCH] D26138: [clang-tidy] Add modernize-use-delete check

Malcolm Parsons via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 31 09:14:58 PDT 2016


malcolm.parsons added a comment.

In https://reviews.llvm.org/D26138#583561, @alexfh wrote:

> "Use delete" makes me think of the `delete` operator. I'd suggest one of these names:


I chose modernize-use-delete as there's already modernize-use-default.
"Use default" makes me think of the `default` case in a switch.

> - modernize-use-equals-delete

Works for me, but should I rename modernize-use-default?


https://reviews.llvm.org/D26138





More information about the cfe-commits mailing list