aboutsummaryrefslogtreecommitdiff
path: root/.clang-tidy
diff options
context:
space:
mode:
authorSulfurNitride <SulfurNitride@users.noreply.github.com>2026-04-28 21:56:22 -0500
committerSulfurNitride <SulfurNitride@users.noreply.github.com>2026-04-28 21:56:22 -0500
commit33015e0393b3aef4a6e129dec5bcf92444321e9c (patch)
treefea7177df0712d09f7d94d24ede877b7e65c56ab /.clang-tidy
parent5d42085dd36f14aa3385547947ccd6e6c5f03d50 (diff)
Add clang-tidy lint infrastructure (.clang-tidy + lint.sh)
`./lint.sh` runs clang-tidy inside the fluorine-builder Docker image so the toolchain matches what compile_commands.json was generated with. Strips -mno-direct-extern-access (gcc-only) before invoking clang and skips browserview/browserdialog since QtWebEngine is Windows-only in this fork. Scope: src/src/ + libs/skse_log_redirector/ (everything we authored). Vendored upstream libs follow their own coding standards and stay out-of-scope. Default-off the noisiest categories (fuchsia/google/llvm/altera, pro-type-*, magic-numbers, named-parameter, function-cognitive- complexity, identifier-length, etc.) so the surfaced warnings are actionable. Dockerfile: install clang-tidy alongside the build deps so the image ships everything lint.sh needs. Usage: ./build.sh tarball # build the image (one-time after this commit) # and produce compile_commands.json ./lint.sh # lint everything ./lint.sh --fix # apply auto-fixes ./lint.sh src/src/foo.cpp # lint a single file Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Diffstat (limited to '.clang-tidy')
-rw-r--r--.clang-tidy66
1 files changed, 66 insertions, 0 deletions
diff --git a/.clang-tidy b/.clang-tidy
new file mode 100644
index 0000000..ce05d5f
--- /dev/null
+++ b/.clang-tidy
@@ -0,0 +1,66 @@
+# Clang-tidy configuration for the Fluorine Manager Linux port.
+#
+# Scope: only files under src/src/ and any libs/ subdirectory we authored
+# (skse_log_redirector). Vendored upstream code (libs/uibase, libs/game_*,
+# libs/installer_fomod*, libs/preview_*, libs/check_fnis, libs/tool_*,
+# libs/bsa*, libs/esptk, libs/archive, libs/libbsarch, libs/usvfs,
+# libs/plugin_python, libs/basic_games, libs/script_extender_plugin_checker,
+# libs/form43_checker, libs/diagnose_basic, libs/dds-header, libs/cmake_common,
+# libs/vcpkg-registry, libs/lzokay.py, libs/winreg.py, libs/7zip, libs/lootcli,
+# libs/bsapacker, libs/bsa_extractor, libs/bsa_ffi, libs/steam_appinfo_ffi)
+# follows upstream MO2 / external projects and is excluded by lint.sh.
+#
+# Disabled checks (with rationale):
+# - fuchsia / google / llvm / altera: house-style checks for projects that
+# aren't this one.
+# - cppcoreguidelines-pro-* / -avoid-c-arrays / -avoid-magic-numbers: too
+# noisy for code that interfaces with Qt and Win32-shaped APIs.
+# - modernize-use-trailing-return-type: stylistic preference.
+# - readability-magic-numbers / -named-parameter / -function-cognitive-
+# complexity / -identifier-length: noisy and largely subjective.
+# - bugprone-easily-swappable-parameters / -branch-clone: too many false
+# positives in plumbing code.
+# - misc-include-cleaner / -no-recursion / -non-private-member-variables-
+# in-classes: noisy and partly redundant with clangd.
+Checks: >
+ bugprone-*,
+ performance-*,
+ modernize-*,
+ readability-*,
+ clang-analyzer-*,
+ misc-const-correctness,
+ misc-redundant-expression,
+ misc-throw-by-value-catch-by-reference,
+ misc-unused-parameters,
+ -modernize-use-trailing-return-type,
+ -modernize-avoid-c-arrays,
+ -modernize-use-nodiscard,
+ -readability-magic-numbers,
+ -readability-named-parameter,
+ -readability-function-cognitive-complexity,
+ -readability-identifier-length,
+ -readability-implicit-bool-conversion,
+ -readability-uppercase-literal-suffix,
+ -readability-braces-around-statements,
+ -readability-else-after-return,
+ -bugprone-easily-swappable-parameters,
+ -bugprone-branch-clone,
+ -bugprone-narrowing-conversions,
+ -performance-no-int-to-ptr,
+ -performance-avoid-endl
+
+WarningsAsErrors: ''
+
+# Apply checks only to our own headers — the regex matches src/src and our
+# authored libs subdirs. Vendored code lives under different paths.
+HeaderFilterRegex: '(^|/)(src/src/|libs/skse_log_redirector/)'
+
+FormatStyle: file
+
+CheckOptions:
+ - key: readability-identifier-naming.ClassCase
+ value: CamelCase
+ - key: readability-identifier-naming.PrivateMemberPrefix
+ value: m_
+ - key: misc-unused-parameters.IgnoreVirtual
+ value: 'true'