C auto-indent do not respect tab width.

[hakan-demirli]

Summary

Minimal reproducable example for #4556 since the issue is not fixed.

Indentation is fixed to 2 spaces even though it is specified as 4.

Reproduction Steps

languages.toml

[[language]]
name = "cpp"
# config = { "clangd.fallbackFlags" = ["-std=c++20"] }
language-server = { command = "clangd", args = [
  "--compile-commands-dir=./build",
] }
auto-format = true
indent = { tab-width = 4, unit = "    " }

main.cpp:

#include <iostream>

int main(int argc, char *argv[]) {
  std::cout << "test";
  return 0;
}

clang version 16.0.6
helix version

Helix log

No response

Platform

Arch Linux

Terminal Emulator

kitty 0.29.2

Helix Version

helix 23.05

[pascalkuthe]

that is not a bug but intended behaviour. Helix auto detects the existing indenation in a file and will use that. The config option is only used for new files or if auto detection fails.

You example is already indented with two spaces so that is what helix detects.

[hakan-demirli]

Auto detection is broken then. Here is a zero indentation example. As soon as file is saved it is indented to two spaces, not four.

#include <iostream>

int main(int argc, char *argv[]) {
std::cout << "test";
return 0;
}

[kirawi]

If it only happens when it's saved, then it's the auto formatter. I think 2 spaces is the default for clangd.

[kirawi]

Also see clangd/clangd#26

[hakan-demirli]

Thank you for the help. It is indeed a clangd problem.

---

This one works.

language-server = { command = "clangd", args = ['--fallback-style=Webkit'] }

This one does not.

  '--fallback-style="{BasedOnStyle: Google, IndentWidth: 4}"',

I run the command standalone and it fails too.

clangd --fallback-style="{BasedOnStyle: Google, IndentWidth: 4}" --check=./main.cpp 

main.cpp: Invalid fallback style "{BasedOnStyle: Google, IndentWidth: 4}. Fallback is LLVM style.

So, no problem on helix side.

[kirawi]

From what I understand that's a clangd bug: clangd/clangd#362