Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mention ccls and clangd in README.md #94

Closed
wants to merge 1 commit into from

Conversation

MaskRay
Copy link
Contributor

@MaskRay MaskRay commented Aug 31, 2018

Whether they work better in lsp-mode is a QoI issue that is unrelated to this PR.

I have checked that both ccls and clangd (built from llvm+clang+extra trunk) work: textDocument/definition.

@MaskRay
Copy link
Contributor Author

MaskRay commented Aug 31, 2018

It's true that I want to promote ccls

@joaotavora joaotavora closed this in f3f8531 Sep 7, 2018
@joaotavora
Copy link
Owner

So ccls is now the default. I tested it and it works decently well without any extra configuration. It's a pity that it still has this abandoned serious cquery bug though :-(

@MaskRay MaskRay deleted the ccls branch September 8, 2018 06:09
@MaskRay
Copy link
Contributor Author

MaskRay commented Sep 8, 2018

Implemented index.onChange: true in git HEAD. I didn't implement this when I rewrote the pipeline.

https://github.com/MaskRay/ccls/blob/master/src/config.h#L204

This has very bad performance impact and currently the cache file will be saved every time a change is made.

@joaotavora
Copy link
Owner

Implemented index.onChange: true in git HEAD. I didn't implement this when I rewrote the pipeline.

I don't have time to test now, but if I remember correctly (and if you read the bug I linked to) this isn't solved by index.onChange: true

@MaskRay
Copy link
Contributor Author

MaskRay commented Sep 8, 2018

I have just added support for empty cacheDirectory (default is ".ccls-cache")

{
  "cacheDirectory": ""
  "index": {
    "onChange": true
  }
}

This is on-the-fly indexing without saving cache files.

bhankas pushed a commit to bhankas/emacs that referenced this pull request Sep 18, 2022
* README.md (Installation and usage): Prefer ccls to cquery. Mention clangd.

* eglot.el (eglot-server-programs): Suggest ccls for c/c++ by default.
bhankas pushed a commit to bhankas/emacs that referenced this pull request Sep 19, 2022
* README.md (Installation and usage): Prefer ccls to cquery. Mention clangd.

* eglot.el (eglot-server-programs): Suggest ccls for c/c++ by default.
bhankas pushed a commit to bhankas/emacs that referenced this pull request Sep 19, 2022
* README.md (Installation and usage): Prefer ccls to cquery. Mention clangd.

* eglot.el (eglot-server-programs): Suggest ccls for c/c++ by default.

#94: joaotavora/eglot#94
jollaitbot pushed a commit to sailfishos-mirror/emacs that referenced this pull request Oct 12, 2022
* README.md (Installation and usage): Prefer ccls to cquery. Mention clangd.

* eglot.el (eglot-server-programs): Suggest ccls for c/c++ by default.

GitHub-reference: close joaotavora/eglot#94
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants