Fix clippy lints

[mkrasnitski]

Ran cargo clippy with default lints. A few small cleanups included as well.

[emesare]

I find the before to be more readable

[mkrasnitski]

yeah, it would be best if the definition of BNInstructionInfo was the following instead:

pub struct BNInstructionInfo {
    pub length: usize,
    pub branchCount: usize,
    pub archTransitionByTargetAddr: bool,
    pub delaySlots: u8,
    pub branchInfo: [BNBranchInfo; 3usize],
}

pub struct BNBranchInfo {
    pub branchType: BNBranchType,
    pub branchTarget: u64,
    #[doc = " If null, same architecture as instruction"]
    pub branchArch: *mut BNArchitecture,
}

Then we could iterate over all the branch info at once instead of by-index. This would require changing the C++ definition though.

[emesare]

Merged with 3ea9c4e thank you! I also went ahead and added clippy to the CI, it should only warn however. Mostly just good for visibility in PR's!