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