Bump clippy to and fix some new warnings

Author: webmaster128

.circleci/config.yml

@@ -71,7 +71,7 @@ workflows:
           matrix:
             parameters:
               # Run with MSRV and some modern stable Rust
-              rust-version: ["1.60.0", "1.65.0"]
+              rust-version: ["1.60.0", "1.66.0"]
       - benchmarking:
           requires:
             - package_vm

contracts/crypto-verify/src/ethereum.rs

@@ -19,7 +19,7 @@ pub fn verify_transaction(
 ) -> StdResult<bool> {
     let sign_bytes =
         serialize_unsigned_transaction(to, nonce, gas, gas_price, value, data, chain_id);
-    let hash = Keccak256::digest(&sign_bytes);
+    let hash = Keccak256::digest(sign_bytes);
     let mut rs: Vec<u8> = Vec::with_capacity(64);
     rs.resize(32 - r.len(), 0); // Left pad r to 32 bytes
     rs.extend_from_slice(r);

packages/crypto/benches/main.rs

@@ -78,7 +78,7 @@ fn bench_crypto(c: &mut Criterion) {
 
     group.bench_function("secp256k1_verify", |b| {
         let message = hex::decode(COSMOS_SECP256K1_MSG_HEX).unwrap();
-        let message_hash = Sha256::digest(&message);
+        let message_hash = Sha256::digest(message);
         let signature = hex::decode(COSMOS_SECP256K1_SIGNATURE_HEX).unwrap();
         let public_key = base64::decode(COSMOS_SECP256K1_PUBKEY_BASE64).unwrap();
         b.iter(|| {

packages/crypto/src/secp256k1.rs

@@ -253,7 +253,7 @@ mod tests {
             let signature = hex::decode(sig).unwrap();
 
             // Explicit hash
-            let message_hash = Sha256::digest(&message);
+            let message_hash = Sha256::digest(message);
 
             // secp256k1_verify works
             assert!(
@@ -290,7 +290,7 @@ mod tests {
             let message = hex::decode(&encoded.message).unwrap();
 
             let hash = hex::decode(&encoded.message_hash).unwrap();
-            let message_hash = Sha256::digest(&message);
+            let message_hash = Sha256::digest(message);
             assert_eq!(hash.as_slice(), message_hash.as_slice());
 
             let signature = hex::decode(&encoded.signature).unwrap();

packages/vm/src/cache.rs

@@ -336,7 +336,7 @@ fn save_wasm_to_disk(dir: impl Into<PathBuf>, wasm: &[u8]) -> VmResult<Checksum>
     // calculate filename
     let checksum = Checksum::generate(wasm);
     let filename = checksum.to_hex();
-    let filepath = dir.into().join(&filename);
+    let filepath = dir.into().join(filename);
 
     // write data to file
     // Since the same filename (a collision resistent hash) cannot be generated from two different byte codes
@@ -561,7 +561,7 @@ mod tests {
             .path()
             .join(STATE_DIR)
             .join(WASM_DIR)
-            .join(&checksum.to_hex());
+            .join(checksum.to_hex());
         let mut file = OpenOptions::new().write(true).open(filepath).unwrap();
         file.write_all(b"broken data").unwrap();
 

packages/vm/src/imports.rs

@@ -225,6 +225,7 @@ pub fn do_secp256k1_verify<A: BackendApi, S: Storage, Q: Querier>(
     let gas_info = GasInfo::with_cost(env.gas_config.secp256k1_verify_cost);
     process_gas_info::<A, S, Q>(env, gas_info)?;
     let result = secp256k1_verify(&hash, &signature, &pubkey);
+    #[allow(clippy::bool_to_int_with_if)]
     Ok(result.map_or_else(
         |err| match err {
             CryptoError::InvalidHashFormat { .. }
@@ -285,6 +286,7 @@ pub fn do_ed25519_verify<A: BackendApi, S: Storage, Q: Querier>(
     let gas_info = GasInfo::with_cost(env.gas_config.ed25519_verify_cost);
     process_gas_info::<A, S, Q>(env, gas_info)?;
     let result = ed25519_verify(&message, &signature, &pubkey);
+    #[allow(clippy::bool_to_int_with_if)]
     Ok(result.map_or_else(
         |err| match err {
             CryptoError::InvalidPubkeyFormat { .. }
@@ -588,7 +590,7 @@ mod tests {
         let result = do_db_read(&env, key_ptr);
         let value_ptr = result.unwrap();
         assert!(value_ptr > 0);
-        assert_eq!(force_read(&env, value_ptr as u32), VALUE1);
+        assert_eq!(force_read(&env, value_ptr), VALUE1);
     }
 
     #[test]

packages/vm/src/modules/file_system_cache.rs

@@ -106,7 +106,7 @@ impl FileSystemCache {
         let filename = checksum.to_hex();
         let file_path = self.latest_modules_path().join(filename);
 
-        let result = unsafe { Module::deserialize_from_file(store, &file_path) };
+        let result = unsafe { Module::deserialize_from_file(store, file_path) };
         match result {
             Ok(module) => Ok(Some(module)),
             Err(DeserializeError::Io(err)) => match err.kind() {

packages/vm/src/modules/pinned_memory_cache.rs

@@ -52,7 +52,7 @@ impl PinnedMemoryCache {
     /// This is based on the values provided with `store`. No actual
     /// memory size is measured here.
     pub fn size(&self) -> usize {
-        self.modules.iter().map(|(_, module)| module.size).sum()
+        self.modules.values().map(|module| module.size).sum()
     }
 }
 

packages/vm/src/wasm_backend/gatekeeper.rs

@@ -711,7 +711,7 @@ mod tests {
         let mut compiler_config = Cranelift::default();
         compiler_config.push_middleware(deterministic);
         let store = Store::new(&Universal::new(compiler_config).engine());
-        let result = Module::new(&store, &wasm);
+        let result = Module::new(&store, wasm);
         assert!(result.is_ok());
     }
 
@@ -732,7 +732,7 @@ mod tests {
         let mut compiler_config = Cranelift::default();
         compiler_config.push_middleware(deterministic);
         let store = Store::new(&Universal::new(compiler_config).engine());
-        let result = Module::new(&store, &wasm);
+        let result = Module::new(&store, wasm);
         assert!(result
             .unwrap_err()
             .to_string()
@@ -759,7 +759,7 @@ mod tests {
         let mut compiler_config = Cranelift::default();
         compiler_config.push_middleware(deterministic);
         let store = Store::new(&Universal::new(compiler_config).engine());
-        let result = Module::new(&store, &wasm);
+        let result = Module::new(&store, wasm);
         assert!(result
             .unwrap_err()
             .to_string()

packages/vm/src/wasm_backend/store.rs

@@ -171,7 +171,7 @@ mod tests {
         let serialized = {
             let wasm = wat::parse_str(EXPORTED_MEMORY_WAT).unwrap();
             let store = make_compile_time_store(None, &[]);
-            let module = Module::new(&store, &wasm).unwrap();
+            let module = Module::new(&store, wasm).unwrap();
             module.serialize().unwrap()
         };