Add warn on `clippy::needless_pass_by_value`

[tcoratger]

As a followup of https://github.com/xJonathanLEI/starknet-rs/pull/598 we should add a warn on this clippy lint

https://rust-lang.github.io/rust-clippy/master/#/needless_pass_by_value?groups=nursery,pedantic

Taking arguments by reference is more flexible and can sometimes avoid unnecessary allocations.