Clippy

Author: grandizzy

crates/compilers/src/cache.rs

@@ -918,7 +918,7 @@ impl<T: ArtifactOutput<CompilerContract = C::CompilerContract>, C: Compiler>
                 return true;
             };
 
-            if entry.interface_repr_hash.as_ref().map_or(true, |h| h != interface_hash) {
+            if entry.interface_repr_hash.as_ref() != Some(interface_hash) {
                 trace!("interface hash changed");
                 return true;
             };
@@ -945,11 +945,11 @@ impl<T: ArtifactOutput<CompilerContract = C::CompilerContract>, C: Compiler>
                 entry.insert(source.content_hash());
             }
             // Fill interface representation hashes for source files
-            if self.is_source_file(&file) {
+            if self.is_source_file(file) {
                 if let hash_map::Entry::Vacant(entry) =
                     self.interface_repr_hashes.entry(file.clone())
                 {
-                    entry.insert(interface_representation_hash(&source, file));
+                    entry.insert(interface_representation_hash(source, file));
                 }
             }
         }

crates/compilers/src/preprocessor.rs

@@ -305,7 +305,7 @@ impl ContractData<'_> {
         let abi_encode_args =
             params.parameters.iter().map(|param| format!("args.{}", param.name)).join(", ");
 
-        let vm_interface_name = format!("VmContractHelper{}", ast_id);
+        let vm_interface_name = format!("VmContractHelper{ast_id}");
         let vm = format!("{vm_interface_name}(0x7109709ECfa91a80626fF3989D68f67F5b1DD12D)");
 
         let helper = format!(
@@ -380,7 +380,7 @@ impl BytecodeDependencyOptimizer<'_> {
         for (path, ast) in &self.asts {
             let src = self.sources.get(path).unwrap().content.as_str();
 
-            if is_test_or_script(path, &self.paths) {
+            if is_test_or_script(path, self.paths) {
                 continue;
             }
 
@@ -428,7 +428,7 @@ impl BytecodeDependencyOptimizer<'_> {
         let mut new_sources = Sources::new();
         for (id, contract) in contracts {
             if let Some(code) = contract.build_helper()? {
-                let path = format!("foundry-pp/DeployHelper{}.sol", id);
+                let path = format!("foundry-pp/DeployHelper{id}.sol");
                 new_sources.insert(path.into(), Source::new(code));
             }
         }
@@ -444,7 +444,7 @@ impl BytecodeDependencyOptimizer<'_> {
         updates: &mut Updates,
     ) -> Result<()> {
         for (path, ast) in &self.asts {
-            if !is_test_or_script(path, &self.paths) {
+            if !is_test_or_script(path, self.paths) {
                 continue;
             }
             let src = self.sources.get(path).unwrap().content.as_str();