Use a constant for checksum length

Author: jkczyz

src/lib.rs

@@ -194,13 +194,13 @@ impl<'a> Bech32Writer<'a> {
         }
 
         // Pad with 6 zeros
-        for _ in 0..6 {
+        for _ in 0..CHECKSUM_LENGTH {
             self.polymod_step(u5(0))
         }
 
         let plm: u32 = self.chk ^ self.variant.constant();
 
-        for p in 0..6 {
+        for p in 0..CHECKSUM_LENGTH {
             self.formatter
                 .write_char(u5(((plm >> (5 * (5 - p))) & 0x1f) as u8).to_char())?;
         }
@@ -468,6 +468,8 @@ pub enum Checksum {
     Exclude,
 }
 
+const CHECKSUM_LENGTH: usize = 6;
+
 /// Encode a bech32 payload to string.
 ///
 /// # Errors
@@ -497,7 +499,7 @@ pub fn encode_without_checksum<T: AsRef<[u5]>>(hrp: &str, data: T, variant: Vari
 /// Returns the HRP in lowercase, the data with the checksum removed, and the encoding.
 pub fn decode(s: &str) -> Result<(String, Vec<u5>, Variant), Error> {
     let (hrp_lower, mut data) = decode_without_checksum(s)?;
-    if data.len() < 6 {
+    if data.len() < CHECKSUM_LENGTH {
         return Err(Error::InvalidLength);
     }
 
@@ -506,7 +508,7 @@ pub fn decode(s: &str) -> Result<(String, Vec<u5>, Variant), Error> {
         Some(variant) => {
             // Remove checksum from data payload
             let dbl: usize = data.len();
-            data.truncate(dbl - 6);
+            data.truncate(dbl - CHECKSUM_LENGTH);
 
             Ok((hrp_lower, data, variant))
         }