diff options
| author | Yuri Astrakhan <YuriAstrakhan@gmail.com> | 2022-12-24 16:09:08 -0500 |
|---|---|---|
| committer | Yuri Astrakhan <YuriAstrakhan@gmail.com> | 2022-12-25 05:07:47 -0500 |
| commit | d3dbf9c194d0badc7c340ae012069f67d55e4753 (patch) | |
| tree | 9f1663fa2108a76e552c5d42dcf91e6a88099aef /tests/ui/iterators/iter-sum-overflow-overflow-checks.rs | |
| parent | 2872e0558982360d499bdba79cf1a3ed06f1ae82 (diff) | |
| download | rust-d3dbf9c194d0badc7c340ae012069f67d55e4753.tar.gz rust-d3dbf9c194d0badc7c340ae012069f67d55e4753.zip | |
Moar linting: needless_borrow, let_unit_value, ...
* There are a few needless borrows that don't seem to be needed. I even did a quick assembly comparison and posted a q to stackoveflow on it. See [here](https://stackoverflow.com/questions/74910196/advantages-of-pass-by-ref-val-with-impl-intoiteratoritem-impl-asrefstr)
* removed several `let _ = ...` when they don't look necessary (even a few ones that were not suggested by clippy (?))
* there were a few `then(|| ctor{})` that clippy suggested to replace with `then_some(ctor{})` -- seems reasonable?
* some unneeded assignment+return - keep the code a bit leaner
* a few `writeln!` instead of `write!`, or even consolidate write!
* a nice optimization to use `ch.is_ascii_digit` instead of `ch.is_digit(10)`
Diffstat (limited to 'tests/ui/iterators/iter-sum-overflow-overflow-checks.rs')
0 files changed, 0 insertions, 0 deletions