]> Untitled Git - bdk/commitdiff
bdk_wallet: descriptor: remove unnecessary call to calc_checksum_bytes
authorAntoine Poinsot <darosior@protonmail.com>
Thu, 25 Jul 2024 14:36:55 +0000 (16:36 +0200)
committerAntoine Poinsot <darosior@protonmail.com>
Sat, 27 Jul 2024 15:56:01 +0000 (17:56 +0200)
It can just be a String.

crates/wallet/src/descriptor/checksum.rs
crates/wallet/src/descriptor/mod.rs

index 243376bca26f17e6dcf03de2d3056660f5f313a9..3a16c3e88cea7ac64b2832b12f0203f073cdd363 100644 (file)
@@ -43,7 +43,7 @@ fn poly_mod(mut c: u64, val: u64) -> u64 {
 }
 
 /// Compute the checksum bytes of a descriptor, excludes any existing checksum in the descriptor string from the calculation
-pub fn calc_checksum_bytes(mut desc: &str) -> Result<[u8; 8], DescriptorError> {
+fn calc_checksum_bytes(mut desc: &str) -> Result<[u8; 8], DescriptorError> {
     let mut c = 1;
     let mut cls = 0;
     let mut clscount = 0;
index 477b265e2fac8cb66abf00c4bcec3d1cc8650f88..c3dd95da8988e8ef4a1e83f7a1d166d442107111 100644 (file)
@@ -42,7 +42,6 @@ pub mod policy;
 pub mod template;
 
 pub use self::checksum::calc_checksum;
-use self::checksum::calc_checksum_bytes;
 pub use self::error::Error as DescriptorError;
 pub use self::policy::Policy;
 use self::template::DescriptorTemplateOut;
@@ -88,8 +87,8 @@ impl IntoWalletDescriptor for &str {
     ) -> Result<(ExtendedDescriptor, KeyMap), DescriptorError> {
         let descriptor = match self.split_once('#') {
             Some((desc, original_checksum)) => {
-                let checksum = calc_checksum_bytes(desc)?;
-                if original_checksum.as_bytes() != checksum {
+                let checksum = calc_checksum(desc)?;
+                if original_checksum != checksum {
                     return Err(DescriptorError::InvalidDescriptorChecksum);
                 }
                 desc