[clang-tools-extra] Follow style configuration in clangd when inserting missing includes (PR #140594)

Nathan Ridge via cfe-commits cfe-commits at lists.llvm.org
Sun May 25 23:03:11 PDT 2025


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


More information about the cfe-commits mailing list