[PATCH] D52028: [clangd] Cleanup FuzzyFindRequest filtering limit semantics

Kirill Bobyrev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 13 05:53:10 PDT 2018


kbobyrev created this revision.
kbobyrev added reviewers: ioeric, ilya-biryukov, sammccall.
kbobyrev added a project: clang-tools-extra.
Herald added subscribers: kadircet, arphaman, jkorous, MaskRay.

As discussed during https://reviews.llvm.org/D51860 review, it is better to use `llvm::Optional` here as it has clear semantics which reflect intended behavior.


https://reviews.llvm.org/D52028

Files:
  clang-tools-extra/clangd/FindSymbols.cpp
  clang-tools-extra/clangd/index/Index.h
  clang-tools-extra/clangd/index/MemIndex.cpp
  clang-tools-extra/clangd/index/dex/Dex.cpp
  clang-tools-extra/unittests/clangd/DexTests.cpp
  clang-tools-extra/unittests/clangd/IndexTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52028.165261.patch
Type: text/x-patch
Size: 7357 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180913/23448901/attachment-0001.bin>


More information about the cfe-commits mailing list