| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | maint: Run maint/add_warning to deny string slices | Clara Engler | 2026-06-09 | 1 | -0/+1 |
| | | | | | | | | | | | | | This commit executes maint/add_warning with the just added change to deny string slices except in tests. I recommend auditing this by checking out the previous commit followed by running the script yourself and then verifying that the diff is identical to this commit. This commit makes cargo clippy fail. We will add exceptions in the next commit. | ||||
| * | proto: Remove an allow for a now-pedantic warning. | Nick Mathewson | 2026-05-07 | 1 | -6/+0 |
| | | |||||
| * | Fix name of clippy lint to unchecked_time_subtraction (2) | Ian Jackson | 2025-11-06 | 1 | -1/+1 |
| | | | | | Run maint/add_warning | ||||
| * | tor-proto: bring back comparison chain w/ allow | hashcatHitman | 2025-10-08 | 1 | -9/+14 |
| | | | | | | | | | It was decided that the comparison chain was actually preferrable for readability. So instead, we're just `allow`ing it until it stops being a problem. Signed-off-by: hashcatHitman <[email protected]> | ||||
| * | tor-proto: replace comparison chain with match | hashcatHitman | 2025-10-08 | 1 | -8/+9 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | There was a comparison chain in `tor_proto::util::poll_all::test::ResolveAfter::poll` which was causing a clippy warning. The lint in question, `clippy::comparison_chain`, was a `clippy::style` lint in 1.85.1 and got moved to `clippy::pedantic` in 1.87.0 (see [rust-clippy!14219]). Since some of us (like me) develop on MSRV, I'm fixing this lint now. Gabi didn't have any strong opinions on whether I did it like this or with an `allow` attribute, so I decided this was better since it means we don't have to come back later just to remove the `allow`. It should be noted that using a match like this can sometimes be a performance regression (see [rust-clippy#5354] and [rust-clippy!6390]). I would expect in this case the effect will be very little, if any, but if tests in `tor_proto::util::poll_all::test` start taking much longer and having an impact on CI or something, this could be why. [rust-clippy!14219]: https://github.com/rust-lang/rust-clippy/pull/14219 [rust-clippy#5354]: https://github.com/rust-lang/rust-clippy/issues/5354 [rust-clippy!6390]: https://github.com/rust-lang/rust-clippy/pull/6390 Signed-off-by: hashcatHitman <[email protected]> | ||||
| * | proto: Document exactly how PollAll polls its futures. | Gabriela Moldovan | 2025-09-25 | 1 | -1/+11 |
| | | |||||
| * | proto: Resolve a couple of clippy warnings in the tests. | Gabriela Moldovan | 2025-09-25 | 1 | -2/+2 |
| | | |||||
| * | proto: Add PollAll helper for driving futures in lockstep. | Gabriela Moldovan | 2025-09-25 | 1 | -0/+195 |
