forked from lix-project/lix
Add basic clang-tidy config
This has not yet had all the warnings Obliterated, but it is a start and
is not *super* far away from being able to run the current configuration
in CI, which will catch some limited number of mistakes.
I tried the meson clang-tidy target and it seems to fail to find flags
for several files, which seems broken. Unsure what is up with that, but
we can use run-clang-tidy or other tooling instead.
We have an extremely annoying situation with the lexer table, which
means that the lexer probably must be moved to another directory with
its own .clang-tidy file to disable the lints in it, *or* write scuffed
code that prepends a disable comment to the top of the generated file.
None of the comment-based lint disabling features work since yacc dumps
a bunch of non compliant code at the top of the file before anything the
user can control.
Change-Id: I1d2aa6ec32deb1db1fbd581127334db1b972323c
This commit is contained in:
parent
5a54b0a20c
commit
99f159c536
18
.clang-tidy
Normal file
18
.clang-tidy
Normal file
|
@ -0,0 +1,18 @@
|
||||||
|
UseColor: true
|
||||||
|
Checks:
|
||||||
|
- -*
|
||||||
|
- bugprone-*
|
||||||
|
# too many warnings
|
||||||
|
- -bugprone-assignment-in-if-condition
|
||||||
|
# too many warnings
|
||||||
|
- -bugprone-narrowing-conversions
|
||||||
|
# kind of nonsense
|
||||||
|
- -bugprone-easily-swappable-parameters
|
||||||
|
# too many warnings for now
|
||||||
|
- -bugprone-implicit-widening-of-multiplication-result
|
||||||
|
# Lix's exception handling is Questionable
|
||||||
|
- -bugprone-empty-catch
|
||||||
|
# many warnings
|
||||||
|
- -bugprone-unchecked-optional-access
|
||||||
|
# many warnings, seems like a questionable lint
|
||||||
|
- -bugprone-branch-clone
|
Loading…
Reference in a new issue