danielmarjamaki added a comment.
In http://reviews.llvm.org/D12359#304772, @danielmarjamaki wrote:
> Moved warning to clang-tidy.
>
> In this patch I am more careful about function calls. Sometimes when it is
> technically possible to use const it's still not a good idea.
>
> For instance when using the standard strstr C function:
>
> char * strstr ( const char *, const char * );
>
> void f(char *p) { // <- p should not be const
> char *a = strstr(p, "a");
> *a = 0;
> }
I intend to fix this also later. But I would prefer to commit this checker
first.
Right now there are not FP.
http://reviews.llvm.org/D12359
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
