[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