JVApen wrote: Thanks for the info. I think is is reasonable to make this change. Clangd already favors C++ over C, for example in the includes to standard library it provides. So moving C to the back of the list seems like a reasonable thing to do.
Throwing an alternative in the mix, maybe we should check for all file extensions. If we find more than 1 file matching stem.ext (excluding current file), we should bypass the file based approach. That way, the symbol approach will be taken and you will correct switching behavior for both stem.c to stem.h and stem.cpp to stem.hpp (feel free to not implement this if you think it's wrong) https://github.com/llvm/llvm-project/pull/198152 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
