[PATCH] D90134: [clangd] Increase the TooMany limit for index-based textual navigation to 5

Nathan Ridge via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 26 22:49:36 PDT 2020


This revision was automatically updated to reflect the committed changes.
Closed by commit rG245b61a330ab: [clangd] Increase the TooMany limit for index-based textual navigation to 5 (authored by nridge).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90134

Files:
  clang-tools-extra/clangd/XRefs.cpp


Index: clang-tools-extra/clangd/XRefs.cpp
===================================================================
--- clang-tools-extra/clangd/XRefs.cpp
+++ clang-tools-extra/clangd/XRefs.cpp
@@ -488,8 +488,8 @@
       Located.Definition = *MaybeDefLoc;
     }
 
-    if (ScoredResults.size() >= 3) {
-      // If we have more than 3 results, don't return anything,
+    if (ScoredResults.size() >= 5) {
+      // If we have more than 5 results, don't return anything,
       // as confidence is too low.
       // FIXME: Alternatively, try a stricter query?
       TooMany = true;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D90134.300895.patch
Type: text/x-patch
Size: 578 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20201027/eb4a6f39/attachment-0001.bin>


More information about the cfe-commits mailing list