[PATCH] D70052: [clang-tidy] Add misc-mutating-copy check
Eugene Zelenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 11 13:07:36 PST 2019
Eugene.Zelenko added a comment.
In D70052#1741246 <https://reviews.llvm.org/D70052#1741246>, @gbencze wrote:
> In D70052#1740235 <https://reviews.llvm.org/D70052#1740235>, @Eugene.Zelenko wrote:
>
> > If this is CERT rule, why check is not placed in relevant module?
>
>
> To be honest I was hoping for some feedback on this as I wasn't sure what the best place for this check would be. Quite a few CERT rules seem to be implemented in other modules and have cert aliases.
> Do you think this check should be moved there or should I just add an alias?
Misc is just heap of unsorted checks, so CERT is definitely better place.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70052/new/
https://reviews.llvm.org/D70052
More information about the cfe-commits
mailing list