[PATCH] D15332: new clang-tidy checker readability-non-const-parameter

Eugene Zelenko via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 10 10:04:43 PST 2015


Eugene.Zelenko added a comment.

In http://reviews.llvm.org/D15332#306831, @danielmarjamaki wrote:

> In http://reviews.llvm.org/D15332#306097, @Eugene.Zelenko wrote:
>
> > This check partially implements PR19419. Could it be extended to variables?
>
>
> Yes, I don't see any technical reasons why that would not work. However politically, some people like const variables and others don't.


May be implement as separate check with some code sharing if applicable?


http://reviews.llvm.org/D15332





More information about the cfe-commits mailing list