ioeric added a comment.

Thanks for the review!



================
Comment at: clangd/index/Index.h:127
   /// \brief A query string for the fuzzy find. This is matched against 
symbols'
-  /// qualfified names.
+  /// qualified names. If any scope below is provided, \p Query is only matched
+  /// against symbols in the scope (excl. symbols in nested scopes).
----------------
sammccall wrote:
> If i'm reading this right, you still support fuzzy-matching the scopes (scope 
> information can be in Query, or Scopes, or both). Why? Semantics would be 
> simpler if query matched the identifier only.
This makes a lot of sense. 


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D41367



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to