[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