[PATCH] D61120: [clangd] Optimize "don't include me" check.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 25 03:29:51 PDT 2019


sammccall added inline comments.


================
Comment at: clangd/index/SymbolCollector.cpp:669
+    return false;
+  return Line.contains_lower("includ");
+}
----------------
ilya-biryukov wrote:
> A typo? Should it be `include`.
it's meant to match "include" or "including". I'll add a comment.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D61120





More information about the cfe-commits mailing list