3939# Fine Tuning
4040# -----------
4141#
42- # - readability-function-cognitive-complexity.IgnoreMacros: 'true':
43- # Otherwise the use of `DEBUG()...` will be penalized, even though it
44- # helps both in debugging and to document the code.
42+ # - readability-function-cognitive-complexity.IgnoreMacros: `true`:
43+ # Otherwise the use of `DEBUG()` will be penalized, even though it
44+ # helps both with debugging and with documenting the code.
45+ # - readability-identifier-length:
46+ # Disabled because of too many false positives.. E.g., `fd` for file
47+ # descriptors or `id` for IDs will be flagged, but are sensible variables
48+ # names when the scope is the function body
49+ # - readability-magic-numbers:
50+ # Disabled because of too many false positives. E.g. in
51+ # ```
52+ # size_t byte_pos = bitpos / 8;
53+ # size_t byte_mask = 1U << (bitpos & 7);
54+ # ```
55+ # the numbers `8` and `7` are considered to be magic, but the code is
56+ # still self-explanatory. Adding `#define BITS_PER_BYTE 8` and
57+ # `#define BIT_POS_IN_BYTE_MASK 7` to silence the warning would not
58+ # improve the readability here, but instead reduce it.
4559#
4660# Warning
4761# -------
@@ -60,6 +74,7 @@ Checks: "clang-analyzer-*,
6074 -bugprone-reserved-identifier,
6175 -bugprone-easily-swappable-parameters,
6276 -readability-identifier-length,
77+ -readability-magic-numbers
6378 "
6479WarningsAsErrors : " bugprone-*,
6580 portability-*,
0 commit comments