[PATCH] D147044: [clangd] Implement cross reference request for #include lines.

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 18 01:38:16 PDT 2023


kadircet added inline comments.


================
Comment at: clang-tools-extra/clangd/IncludeCleaner.h:87
+std::optional<include_cleaner::Header>
+firstMatchedProvider(const include_cleaner::Includes &Includes,
+                     llvm::ArrayRef<include_cleaner::Header> Providers);
----------------
VitaNuo wrote:
> kadircet wrote:
> > can you also move tests related to this functionality from HoverTests into IncludeCleanerTests ?
> Not really, not sure what you mean.
> 
> This function receives a ranked list of providers that should be a result of include_cleaner analysis.
> 
> If I just test the function based on a fake list of providers, the test will be trivial. The current hover test seems to provide more value than that.
> If I just test the function based on a fake list of providers, the test will be trivial.

but that's the "unit" of logic we're trying to test, no?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D147044/new/

https://reviews.llvm.org/D147044



More information about the cfe-commits mailing list