HighCommander4 wrote:

> > But since we have both config options, maybe there is a reason for 
> > overwrites in this direction as well.
> 
> Hmm, I was mainly focusing on the quoted -> angled case, hence why only the 
> `AngledHeaders` filter is used. But I have to agree that following the config 
> fully makes sense. I see other instances, like 
> `IncludeInserter::calculateIncludePath` do make use of both filters. I can 
> make the change, to overwrite the inclusion style when the configs dictate 
> that, if that is the desired behavior here.

+1, it makes sense to check both filters. Thanks Kadir for catching that.

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

Reply via email to