New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Clippy easy wins #596

Merged

quentinlesceller merged 1 commit into mimblewimble:master from trevyn:trevyn-clippy1-redo

Mar 17, 2021

Contributor

trevyn commented Mar 17, 2021

To make these easy to review, these are nice obvious wins that make the code smaller and cleaner. :)

All of the removed clone()s are unnecessary, usually because that type implements Copy.

cargo test --all passes locally.


          Clippy easy wins 1

bd92caf

quentinlesceller approved these changes

View reviewed changes

Member

quentinlesceller left a comment

Looking good! Thank you @trevyn. Will merge once CI pass.

quentinlesceller merged commit a88a522 into mimblewimble:master

trevyn deleted the trevyn-clippy1-redo branch

March 17, 2021 16:13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment