fix build on 1.48.0

Author: apoelstra

contrib/test.sh

@@ -49,8 +49,7 @@ if [ "${DO_FEATURE_MATRIX-false}" = true ]; then
     build_and_test "alloc"
     build_and_test "std alloc"
     # arrayvec breaks the MSRV
-    if [ $MSRV = false
-       ]; then
+    if [ $MSRV = false ]; then
         build_and_test "arrayvec"
         build_and_test "std arrayvec"
         build_and_test "alloc arrayvec"

src/primitives/gf32.rs

@@ -411,8 +411,8 @@ mod tests {
 
     #[test]
     fn from_char() {
-        for c in CHARS_LOWER {
-            assert!(Fe32::from_char(c).is_ok())
+        for c in &CHARS_LOWER[..] {
+            assert!(Fe32::from_char(*c).is_ok())
         }
     }
 
@@ -426,8 +426,8 @@ mod tests {
 
     #[test]
     fn mul_zero() {
-        for c in CHARS_LOWER {
-            let fe = Fe32::from_char(c).unwrap();
+        for c in &CHARS_LOWER[..] {
+            let fe = Fe32::from_char(*c).unwrap();
             assert_eq!(fe._mul(Fe32::Q), Fe32::Q) // Fe32::Q == Fe32(0)
         }
     }
@@ -446,8 +446,8 @@ mod tests {
 
     #[test]
     fn mul_one() {
-        for c in CHARS_LOWER {
-            let fe = Fe32::from_char(c).unwrap();
+        for c in &CHARS_LOWER[..] {
+            let fe = Fe32::from_char(*c).unwrap();
             assert_eq!(fe * Fe32::P, fe) // Fe32::P == Fe32(1)
         }
     }

src/primitives/hrp.rs

@@ -46,23 +46,22 @@ impl Hrp {
 
     /// Parses the human-readable part (see [`Hrp::parse`] for full docs).
     ///
-    /// # Panics
-    ///
-    /// If the string is invalid as defined by BIP-173.
+    /// Does not check that `hrp` is valid according to BIP-173 but does check for valid ASCII
+    /// values, replacing any invalid characters with `X`.
     pub const fn parse_unchecked(hrp: &str) -> Self {
         let mut new = Hrp { buf: [0_u8; MAX_HRP_LEN], size: 0 };
         let hrp_bytes = hrp.as_bytes();
 
         let mut i = 0;
         // Funky code so we can be const.
         while i < hrp.len() {
-            let b = hrp_bytes[i];
+            let mut b = hrp_bytes[i];
             // Valid subset of ASCII
             if b < 33 || b > 126 {
-                panic!("invalid hrp");
+                b = b'X';
             }
 
-            new.buf[i] = hrp_bytes[i];
+            new.buf[i] = b;
             new.size += 1;
             i += 1;
         }