[PATCH] D53933: [clangd] Get rid of QueryScopes.empty() == AnyScope special case.

[Sam McCall via Phabricator via cfe-commits]

sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.

Yay, thanks!



================
Comment at: clangd/index/Index.h:463
   /// The global scope is "", a top level scope is "foo::", etc.
-  /// FIXME: drop the special case for empty list, which is the same as
-  /// `AnyScope = true`.
   /// FIXME: support scope proximity.
   std::vector<std::string> Scopes;
----------------
(while here - is this fixme also addressed?)


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53933