Clean-up clang-tidy reports
This is a rolling issue to monitor clang-tidy reports
+-----------------------------------------------------+---------+
| code | count |
|-----------------------------------------------------+---------|
| cppcoreguidelines-pro-bounds-pointer-arithmetic | 155 |
| cppcoreguidelines-special-member-functions | 104 |
| cppcoreguidelines-pro-bounds-constant-array-index | 69 |
| modernize-pass-by-value | 60 |
| cppcoreguidelines-pro-type-member-init | 40 |
| misc-macro-parentheses | 37 |
| readability-else-after-return | 29 |
| cppcoreguidelines-pro-type-const-cast | 24 |
| misc-noexcept-move-constructor | 22 |
| readability-braces-around-statements | 14 |
| modernize-use-emplace | 14 |
| misc-move-const-arg | 14 |
| performance-unnecessary-copy-initialization | 8 |
| modernize-use-equals-delete | 8 |
| readability-misleading-indentation | 7 |
| cppcoreguidelines-pro-bounds-array-to-pointer-decay | 6 |
| readability-redundant-control-flow | 6 |
| readability-simplify-boolean-expr | 5 |
| misc-redundant-expression | 4 |
| misc-unused-using-decls | 4 |
| readability-avoid-const-params-in-decls | 4 |
| modernize-loop-convert | 2 |
| misc-use-after-move | 2 |
| clang-analyzer-deadcode.DeadStores | 1 |
| misc-definitions-in-headers | 1 |
| readability-redundant-string-init | 1 |
| modernize-raw-string-literal | 1 |
| readability-redundant-smartptr-get | 1 |
| modernize-make-unique | 1 |
| clang-analyzer-core.UndefinedBinaryOperatorResult | 1 |
| misc-suspicious-semicolon | 1 |
| readability-redundant-declaration | 1 |
| performance-inefficient-vector-operation | 1 |
| modernize-use-noexcept | 1 |
| modernize-deprecated-headers | 1 |
| misc-sizeof-expression | 1 |
| modernize-make-shared | 1 |
+-----------------------------------------------------+---------+
The Alternative name of this issue is: "Sometimes I sit in interesting meetings and I decide to not fully pay attention, so at least I can do some clang tidying."