schenker wrote:

> I'm not sure now if this wont cause too much false-positives, after all this 
> is why CheckFunctionCalls option were added. Sometimes object can have 2 same 
> operators, one const and one not, in such case depend on "this" type, 
> non-const could be used, this would be false-positive.

You are right, I misunderstood `CheckFunctionCalls` and it's trickier than I 
thought. Anyways, I think it would make sense to treat method calls and method 
operator calls the same way.

I will rething my imlementation, thanks for reviewing!



https://github.com/llvm/llvm-project/pull/71974
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to