MarcoFalke
faa11434fe
refactor: Enable all clang-tidy plugin bitcoin tests
...
This makes it easier to add new ones without having to modify this file
every time.
2023-08-16 14:48:06 +02:00
fanquake
1c976c691c
tidy: Integrate bicoin-tidy clang-tidy plugin
...
Enable `bitcoin-unterminated-logprintf`.
Co-authored-by: Cory Fields <cory-nospam-@coryfields.com>
2023-08-03 17:52:24 +01:00
fanquake
047daad4f5
clang-tidy: turn on modernize-use-noexcept
2023-07-17 14:45:42 +01:00
Hennadii Stepanov
03ec5b6f9c
clang-tidy: Exclude performance-*
checks rather including them
2023-03-26 20:18:13 +01:00
Hennadii Stepanov
2400437230
clang-tidy: Add performance-type-promotion-in-math-fn
check
...
https://clang.llvm.org/extra/clang-tidy/checks/performance/type-promotion-in-math-fn.html
2023-03-26 20:18:03 +01:00
Hennadii Stepanov
7e975e6cf8
clang-tidy: Add performance-inefficient-vector-operation
check
...
https://clang.llvm.org/extra/clang-tidy/checks/performance/inefficient-vector-operation.html
2023-03-26 20:17:55 +01:00
Hennadii Stepanov
516b75f66e
clang-tidy: Add performance-faster-string-find
check
...
https://clang.llvm.org/extra/clang-tidy/checks/performance/faster-string-find.html
2023-03-26 20:17:46 +01:00
MarcoFalke
fa451d4b60
Fix clang-tidy readability-const-return-type violations
2023-02-01 11:33:35 +01:00
Hennadii Stepanov
b0e916913c
clang-tidy: Force to check all headers
2023-01-31 11:50:24 +00:00
Hennadii Stepanov
06fc29326b
refactor: Remove duplication of clang-tidy's check names
2023-01-17 15:40:29 +00:00
Hennadii Stepanov
b7f6a89a3e
Merge bitcoin-core/gui#686 : clang-tidy: Force checks for headers in src/qt
...
7b7cd11244
clang-tidy, qt: Force checks for headers in `src/qt` (Hennadii Stepanov)
69eacf2c5e
clang-tidy, qt: Fix `modernize-use-default-member-init` in headers (Hennadii Stepanov)
Pull request description:
This PR split from bitcoin/bitcoin#26705 and contains only changes in `src/qt`.
Effectively, it fixes the clang-tidy's `modernize-use-default-member-init` errors, and forces clang-tidy checks for all headers in the `src/qt` directory.
ACKs for top commit:
jarolrod:
ACK 7b7cd11244
Tree-SHA512: 79525bb0f31ae7cad88c781e55091a21467c0485ddc1ed03ad62e051480fda3b3710619ea11af480437edba3c6e038f7c40edc6b373e3a37408c006d11b34686
2023-01-17 09:54:56 +00:00
Hennadii Stepanov
9567bfeab9
clang-tidy: Add performance-no-automatic-move
check
...
https://clang.llvm.org/extra/clang-tidy/checks/performance/no-automatic-move.html
2022-12-27 15:25:51 +00:00
Hennadii Stepanov
7b7cd11244
clang-tidy, qt: Force checks for headers in src/qt
2022-12-16 11:58:46 +00:00
fanquake
01e1627e25
Merge bitcoin/bitcoin#25872 : Fix issues when calling std::move(const&)
...
fa875349e2
Fix iwyu (MacroFake)
faad673716
Fix issues when calling std::move(const&) (MacroFake)
Pull request description:
Passing a symbol to `std::move` that is marked `const` is a no-op, which can be fixed in two ways:
* Remove the `const`, or
* Remove the `std::move`
ACKs for top commit:
ryanofsky:
Code review ACK fa875349e2
. Looks good. Good for univalue to support c++11 move optimizations
Tree-SHA512: 3dc5cad55b93cfa311abedfb811f35fc1b7f30a1c68561f15942438916c7de25e179c364be11881e01f844f9c2ccd71a3be55967ad5abd2f35b10bb7a882edea
2022-08-31 08:38:24 +01:00
fanquake
f345dc3960
tidy: enable bugprone-use-after-move
...
Will error with:
```bash
coins.cpp:102:22: error: 'coin' used after it was moved [bugprone-use-after-move,-warnings-as-errors]
(uint32_t)coin.nHeight,
^
coins.cpp:96:21: note: move occurred here
it->second.coin = std::move(coin);
```
until #25663 is merged.
See:
https://releases.llvm.org/14.0.0/tools/clang/tools/extra/docs/clang-tidy/checks/bugprone-use-after-move.html
2022-08-30 15:19:53 +01:00
MacroFake
faad673716
Fix issues when calling std::move(const&)
2022-08-20 09:32:53 +02:00
Aurèle Oulès
ae7ae36d31
tidy: Enable two clang-tidy checks
...
Checks enabled: 'performance-for-range-copy' and 'performance-unnecessary-copy-initialization'
2022-08-03 17:18:17 +02:00
fanquake
49168df073
tidy: enable readability-redundant-string-init
...
See:
https://releases.llvm.org/14.0.0/tools/clang/tools/extra/docs/clang-tidy/checks/readability-redundant-string-init.html
2022-07-26 10:16:42 +01:00
fanquake
a02f3f19f5
tidy: use misc-unused-using-decls
...
https://clang.llvm.org/extra/clang-tidy/checks/misc/unused-using-decls.html
2022-07-18 17:25:07 +01:00
fanquake
ac6fbf2c83
tidy: use modernize-use-default-member-init
2022-05-17 17:19:07 +01:00
fanquake
c2b295881f
tidy: add readability-redundant-declaration
2022-05-01 10:39:40 +01:00
fanquake
9c96f1008b
tidy: enable modernize-use-nullptr
2022-04-26 10:43:33 +01:00
MarcoFalke
fa57fa1a2e
Enable clang-tidy bugprone-argument-comment and fix violations
2021-09-07 09:11:10 +02:00