Merge #133: Remove Hrp::is_empty

0909bec Remove Hrp::is_empty (Tobin C. Harding)

Pull request description:

  This function always returns `false`, it is pointless - remove it.

ACKs for top commit:
  apoelstra:
    ACK 0909bec

Tree-SHA512: ab36cc32aed6658894323cda726bf0fd30c22ede7f92dba7480e49692fdbf566741a48ad7842e84859f12e583c4cf17e2c0a203b99e967b014b62ab24172341d

Author: apoelstra

src/primitives/hrp.rs

@@ -178,12 +178,9 @@ impl Hrp {
     ///
     /// Guaranteed to be between 1 and 83 inclusive.
     #[inline]
+    #[allow(clippy::len_without_is_empty)] // HRP is never empty.
     pub fn len(&self) -> usize { self.size }
 
-    /// Always false, the human-readable part is guaranteed to be between 1-83 characters.
-    #[inline]
-    pub fn is_empty(&self) -> bool { false }
-
     /// Returns `true` if this [`Hrp`] is valid according to the bips.
     ///
     /// [BIP-173] states that the HRP must be either "bc" or "tb".