use ok_or_else instead

varfish-org · Jan 7, 2025 · d700f68 · d700f68
1 parent ca3c944
commit d700f68
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/src/sequences.rs b/src/sequences.rs
@@ -196,10 +196,9 @@ pub fn aa3_to_aa1(seq: &str) -> Result<String, Error> {
     let mut result = String::with_capacity(seq.len() / 3);
 
     for (i, aa3) in seq.as_bytes().chunks(3).enumerate() {
-        let aa1 = _aa3_to_aa1(aa3).ok_or(Error::InvalidThreeLetterAminoAcid(
-            format!("{:?}", aa3),
-            format!("{}", i + 1),
-        ))? as char;
+        let aa1 = _aa3_to_aa1(aa3).ok_or_else(|| {
+            Error::InvalidThreeLetterAminoAcid(format!("{:?}", aa3), format!("{}", i + 1))
+        })? as char;
         result.push(aa1);
     }