This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Store index in '.clangd/index' instead of '.clangd-index'
ClosedPublic

Authored by ilya-biryukov on Feb 20 2019, 4:40 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

ilya-biryukov created this revision.Feb 20 2019, 4:40 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 20 2019, 4:40 AM
kadircet accepted this revision.Feb 20 2019, 8:37 AM

Thanks!

clang-tools-extra/clangd/index/BackgroundIndexStorage.cpp
75 ↗(On Diff #187549)

NIT: it looks like we don't need the "/" at the end of "index/"

This revision is now accepted and ready to land.Feb 20 2019, 8:37 AM
ilya-biryukov marked an inline comment as done.
  • Remove trailing slash from the path
ilya-biryukov added inline comments.Feb 20 2019, 10:59 AM
clang-tools-extra/clangd/index/BackgroundIndexStorage.cpp
75 ↗(On Diff #187549)

Done.
I was not sure if it is important or not. Everything appears to work without it, so removing it now.

  • Update .gitignore
Herald added a project: Restricted Project. · View Herald TranscriptFeb 20 2019, 11:04 AM
This revision was automatically updated to reflect the committed changes.