Replace a missed unwrap with an ExpectedConstant error

lopopolo · lopopolo · commit dae66ac74383 · 2020-06-13T19:48:32.000-07:00
diff --git a/src/lib.rs b/src/lib.rs
@@ -233,7 +233,9 @@ pub fn decode<T: AsRef<[u8]>>(encoded: T) -> Result<Vec<u8>, DecodeError> {
                 return Err(DecodeError::ChecksumMismatch);
             }
         } else {
-            let b = CONSONANTS.find_byte(*mid).unwrap() as u8;
+            let b = CONSONANTS
+                .find_byte(*mid)
+                .ok_or(DecodeError::ExpectedConsonant)? as u8;
             decoded.push(decode_3_tuple(a, b, c, checksum)?);
         }
         Ok(decoded)

Original file line number	Diff line number	Diff line change
`@@ -233,7 +233,9 @@ pub fn decode<T: AsRef<[u8]>>(encoded: T) -> Result<Vec<u8>, DecodeError> {`
`233`	`233`	`return Err(DecodeError::ChecksumMismatch);`
`234`	`234`	`}`
`235`	`235`	`} else {`
`236`		`- let b = CONSONANTS.find_byte(*mid).unwrap() as u8;`
	`236`	`+ let b = CONSONANTS`
	`237`	`+ .find_byte(*mid)`
	`238`	`+ .ok_or(DecodeError::ExpectedConsonant)? as u8;`
`237`	`239`	`decoded.push(decode_3_tuple(a, b, c, checksum)?);`
`238`	`240`	`}`
`239`	`241`	`Ok(decoded)`