alexfh added a comment.

In https://reviews.llvm.org/D41716#967247, @vmiklos wrote:

> In https://reviews.llvm.org/D41716#967237, @lebedev.ri wrote:
>
> > That changes the defaults though. I thought clang-tidy *tried* to produce 
> > the same results
> >  on different clang-tidy versions with the same `.clang-tidy` config? Or is 
> > there no such guarantees?
>
>
> Hmm, I see this as a trade-off between backwards compatibility and 
> consistency. Here I went for consistency, so that all of 
> readability-inconsistent-declaration-parameter-name, 
> modernize-use-default-member-init and modernize-use-using default to 
> IgnoreMacros=1. But I don't have a too strong opinion on that, can change the 
> default to 0 if really wanted,


I'd personally prefer consistency in this case, especially because the option 
can be specified globally for all checks.


https://reviews.llvm.org/D41716



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to