Paul Smith

Within a double-quoted string backslashes do not need to be escaped unless they are escaping a special character. See the POSIX spec link I put into the code review. So,...

Just to point out, we should probably provide an update to the README that mentions this portability concern for pathnames when talking to Windows servers and suggests a sanitizer function....

Thanks for that pointer. Some comments in the bug and PR imply that you must use POSIX-style paths (forward slashes) with Windows OpenSSH server. In fact, the Windows OpenSSH server...

> clangd has auto-completion mode where it inserts full function signature with placeholders for arguments. I could find ccls option to do that. This might depend on your LSP client....

Is your `/usr/bin/c++` clang, or is it GCC? In other words, are you compiling your actual code with clang, or are you compiling it with GCC?

> I would expect ccls to execute the clang pointed to by compile_commands.json with -print-resource-dir and use that value instead. That wouldn't be correct. ccls does not use the clang...

I agree, the error handling should be better and if these directories are indeed always required then ccls should fail when they don't exist. However I'm not 100% sure it's...

Sounds good. Please also file a bug against whatever package you are using, so that others using that package won't run into this issue. The GitHub ccls project is a...

> My perspective is as follows: if I can execute the commands in compile_commands.json ccls should be smart enough to do so too (if only as hardcoded logic specifically for...

Exactly. As I said, a ccls package _should_ provide those files as part of its package, and the ccls binary in the package _should_ be configured to look for those...