tonarino / actor

A minimalist actor framework aiming for high performance and simplicity.
MIT License
40 stars 6 forks source link

Fix clippy 1.73 lints, update lint configuration #78

Closed strohel closed 8 months ago

strohel commented 8 months ago

timed::QueueItem: Implement partial_cmp() using cmp(), not the other way around

Fixes clippy 1.73 lint.

This different order makes sense, as it avoids an expect().

Clippy lints update: all is implied, add todo

I must have been mistaken when originally introducing clippy:all, because these are, by definition, lints that are enabled without any configuration: https://doc.rust-lang.org/stable/clippy/usage.html#lint-configuration

Add a lint that catches todo!() (but allows panic!(), unreachable!(), unimplemented!()) - you're encouraged to use it during development as a reminder for stuff to finish before a change can be merged.