diff options
| author | bors <bors@rust-lang.org> | 2020-07-26 19:48:17 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2020-07-26 19:48:17 +0000 |
| commit | f5d429cd762423901f946abd800dc2cd91366ccf (patch) | |
| tree | 676a361733f8b6dd8ded9d0230d84d5e28f57aac /src/rustllvm/CoverageMappingWrapper.cpp | |
| parent | da5a6fb1b65ec6581a67e942a3850f6bc15a552c (diff) | |
| parent | 442c8ae23b90874485468b3becfc011f8c9d40bb (diff) | |
| download | rust-f5d429cd762423901f946abd800dc2cd91366ccf.tar.gz rust-f5d429cd762423901f946abd800dc2cd91366ccf.zip | |
Auto merge of #5820 - ThibsG:FixSuspiciousArithmeticImpl, r=flip1995
Fix FP for `suspicious_arithmetic_impl` from `suspicious_trait_impl` … As discussed in #3215, the `suspicious_trait_impl` lint causes too many false positives, as it is complex to find out if binary operations are suspicious or not. This PR restricts the number of binary operations to at most one, otherwise we don't lint. This can be seen as very conservative, but at least FP can be reduced to bare minimum. Fixes: #3215 changelog: limit the `suspicious_arithmetic_impl` lint to one binop, to avoid many FPs
Diffstat (limited to 'src/rustllvm/CoverageMappingWrapper.cpp')
0 files changed, 0 insertions, 0 deletions