search

vmware-archive / haret

A strongly consistent distributed coordination system, built using proven protocols & implemented in Rust.
461 stars 18 forks source link

Address many of the clippy lints #131

Closed erickt closed 7 years ago

erickt commented 7 years ago

https://github.com/manishearth/rust-clippy, if you are not familiar with it, is the community's de facto linter. This implements many of the clippy suggestions on how to make code a bit more ideomatic, and cut down on superfluous copies. There are still a few lints I haven't addressed yet, but we can get that in a future pass.

andrewjstone commented 7 years ago

HI @erickt. Thanks so much for this. I knew about clippy and @jrgarcia actually added a Makefile target for it, I just haven't had an opportunity to fix the warnings. This is super helpful. I'll take a look at this tomorrow. I was busy wrapping up #132 earlier today.

andrewjstone commented 7 years ago

LGTM. Thanks again @erickt