Skip to content

Commit

Permalink
Update clang format in build md (#745)
Browse files Browse the repository at this point in the history
* Update the minimum version of clang-format to version 9

A small regression was created with this commit
6076d74
Variable AfterCaseLabel, was added in clang-format-9, so it is not
supported by version 8

* Update Clang Format version

- Changing the recommended version of clang-forant to version 15
- Changing the minimum version that supports formatting to versions 9,
change for is due to adding AfterCaseLabel in this commit 6076d74.
The AfterCaseLabel modifier was added in clang-format-9.
  • Loading branch information
Dawid-Lorenz-Mobica authored Jul 13, 2023
1 parent 1fa4b01 commit d586556
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion docs/build.md
Original file line number Diff line number Diff line change
Expand Up @@ -161,7 +161,7 @@ adb shell setprop security.perf_harden 0

### Clang Format and Visual Studio

Visual Studio comes with `clang-format-6` which is incompatible with some of the styles we use in our `.clang-format` file. It is recommended to point to a `clang-format-8.exe` binary within the in-built clang formatter, or disable it and use a third party extension that is more up to date.
It is recommended to use `clang-format-15`, which is compatible with the styles in our `.clang-format` file. It is also used by CI and is a basic version installed with Visual Studio 2022. The minimum version that supports our current `.clang-format` file is `clang-format-9`.

Go to the [LLVM downloads page](http://releases.llvm.org/download.html) to get clang.

Expand Down

0 comments on commit d586556

Please sign in to comment.