forked from codilime/veles
-
Notifications
You must be signed in to change notification settings - Fork 0
/
.clang-tidy
28 lines (27 loc) · 1.53 KB
/
.clang-tidy
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
---
# No idea how to fold this line without breaking clang-tidy.
Checks: 'clang-diagnostic-*,clang-analyzer-*,google-*,performace-*,bugprone-*,cert-*,misc-*,readability-*,modernize-*,-cert-err58-cpp,-misc-misplaced-widening-cast,-modernize-pass-by-value,-modernize-use-equals-default,-modernize-raw-string-literal,-modernize-use-equals-delete,-modernize-use-default-member-init,-modernize-use-emplace,-readability-braces-around-statements,-misc-suspicious-enum-usage'
...
# Disabled checks:
# - cert-err58-cpp:
# We use many QString globals which theoretically can throw in constructors
# (and we're fine with that).
# - misc-misplaced-widening-cast:
# Too many warnings when using Qt (e.g. in
# `static_cast<Qt::ItemDataRole>(Qt::UserRole + 1)` ).
# - modernize-use-equals-default:
# Doesn't work well with our Variant-like union in msgpackobject.h.
# - modernize-raw-string-literal
# Proposes changes even for short and trivial strings, e.g. "asd\\asd" (raw
# string literals have long and ugly syntax, not worth to use in such
# places).
# - modernize-use-equals-delete
# Doesn't work with GTest's TEST() macro.
# - modernize-use-default-member-init
# Sometimes it is more readable to set some members in a constructor.
# - modernize-use-emplace
# Warns also about push_back(SomeFactory::create()).
# - readability-braces-around-statements:
# There are parts of the code where short `if(A) B;` is preferred.
# - misc-suspicious-enum-usage:
# Incompatible with Qt (e.g. QKeySequence(Qt::CTRL | Qt::Key_G)).