binaryninja-api icon indicating copy to clipboard operation
binaryninja-api copied to clipboard

Fix clippy warnings and run rustfmt

Open mkrasnitski opened this issue 1 year ago • 1 comments

Clippy was complaning about some uses of std::mem::transmute, so I changed them to use simple as casting instead. Additionally, ran cargo fmt --all to avoid extraneous formatting changes from leaking into future PRs.

mkrasnitski avatar Jul 18 '24 04:07 mkrasnitski

Due to the nature of formatting all of the rust api, we are instead going to format the code in rust_break_everything. Those refactors also will create large diffs, and doing them both at the same time is better than if we were to go ahead and format now, then follow it shortly after with another large refactor.

I am going to keep this PR up until rust_break_everything is merged.

emesare avatar Aug 24 '24 15:08 emesare