search

mozilla / cubeb-pulse-rs

ISC License
16 stars 17 forks source link

Fix clippy warnings #70

Closed ChunMinChang closed 3 years ago

ChunMinChang commented 3 years ago

Fix a couple of clippy warnings

ChunMinChang commented 3 years ago

Should we run cargo clippy in GitHub action to catch some common mistakes in the following PRs?

kinetiknz commented 3 years ago

Should we run cargo clippy in GitHub action to catch some common mistakes in the following PRs?

Yeah, that'd be great. We can also enable cargo fmt checks since existing issues were fixed in c3a51357f9674756311848bf1e5966a0b7c3eef9.

kinetiknz commented 3 years ago

Yeah, that'd be great. We can also enable cargo fmt checks since existing issues were fixed in c3a5135.

Both enabled in ac1e4b5b17eb437de4038ba20bcaf51ec5ccb7f9.