hokein added inline comments.

================
Comment at: clangd/index/dex/dexp/Dexp.cpp:128
   };
   cl::opt<unsigned> Limit{
       "limit",
----------------
ioeric wrote:
> I think we should make it configurable like what we do here.
Made it configurable for `Lookup`. `Limit` option is used to restrict the 
number for final results, it is reasonable for `FuzzyFind`, and `Lookup`

But for `Refs`, it means the number of different symbols, not the number of 
final results, I still hard-code it.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D54106



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

Reply via email to