update descriptor struct

Author: sanket1729

examples/htlc.rs

@@ -19,8 +19,8 @@ extern crate miniscript;
 
 use bitcoin::Network;
 use miniscript::policy::{Concrete, Liftable};
-use miniscript::Descriptor;
 use miniscript::NullCtx;
+use miniscript::{Descriptor, DescriptorTrait};
 use std::str::FromStr;
 
 fn main() {
@@ -32,7 +32,12 @@ fn main() {
                                                   expiry = "4444"
     )).unwrap();
 
-    let htlc_descriptor = Descriptor::Wsh(htlc_policy.compile().unwrap());
+    let htlc_descriptor = Descriptor::new_wsh(
+        htlc_policy
+            .compile()
+            .expect("Policy compilation only fails on resource limits or mixed timelocks"),
+    )
+    .expect("Resource limits");
 
     // Check whether the descriptor is safe
     // This checks whether all spend paths are accessible in bitcoin network.
@@ -62,7 +67,7 @@ fn main() {
     assert_eq!(
         format!(
             "{}",
-            htlc_descriptor.address(Network::Bitcoin, NullCtx).unwrap()
+            htlc_descriptor.address(NullCtx, Network::Bitcoin).unwrap()
         ),
         "bc1qmpfcw7he9z5d9ftfe8qw699azmm2sr8fen903fs4plv007yx0t3qxfmqv5"
     );

examples/parse.rs

@@ -17,7 +17,7 @@
 extern crate bitcoin;
 extern crate miniscript;
 
-use miniscript::NullCtx;
+use miniscript::{DescriptorTrait, NullCtx};
 use std::str::FromStr;
 
 fn main() {

examples/sign_multisig.rs

@@ -18,7 +18,7 @@ extern crate bitcoin;
 extern crate miniscript;
 
 use bitcoin::secp256k1; // secp256k1 re-exported from rust-bitcoin
-use miniscript::NullCtx;
+use miniscript::{DescriptorTrait, NullCtx};
 use std::collections::HashMap;
 use std::str::FromStr;
 

examples/xpub_descriptors.rs

@@ -17,7 +17,7 @@
 extern crate miniscript;
 
 use miniscript::bitcoin::{self, secp256k1};
-use miniscript::{Descriptor, DescriptorPublicKey, DescriptorPublicKeyCtx};
+use miniscript::{Descriptor, DescriptorPublicKey, DescriptorPublicKeyCtx, DescriptorTrait};
 
 use bitcoin::util::bip32;
 
@@ -34,13 +34,13 @@ fn main() {
             "wsh(sortedmulti(1,xpub661MyMwAqRbcFW31YEwpkMuc5THy2PSt5bDMsktWQcFF8syAmRUapSCGu8ED9W6oDMSgv6Zz8idoc4a6mr8BDzTJY47LJhkJ8UB7WEGuduB,xpub69H7F5d8KSRgmmdJg2KhpAK8SR3DjMwAdkxj3ZuxV27CprR9LgpeyGmXUbC6wb7ERfvrnKZjXoUmmDznezpbZb7ap6r1D3tgFxHmwMkQTPH))",
         )
         .unwrap()
-        .address(bitcoin::Network::Bitcoin, desc_ctx).unwrap();
+        .address(desc_ctx, bitcoin::Network::Bitcoin).unwrap();
 
     let addr_two = Descriptor::<DescriptorPublicKey>::from_str(
             "wsh(sortedmulti(1,xpub69H7F5d8KSRgmmdJg2KhpAK8SR3DjMwAdkxj3ZuxV27CprR9LgpeyGmXUbC6wb7ERfvrnKZjXoUmmDznezpbZb7ap6r1D3tgFxHmwMkQTPH,xpub661MyMwAqRbcFW31YEwpkMuc5THy2PSt5bDMsktWQcFF8syAmRUapSCGu8ED9W6oDMSgv6Zz8idoc4a6mr8BDzTJY47LJhkJ8UB7WEGuduB))",
         )
         .unwrap()
-        .address(bitcoin::Network::Bitcoin, desc_ctx).unwrap();
+        .address(desc_ctx, bitcoin::Network::Bitcoin).unwrap();
     let expected = bitcoin::Address::from_str(
         "bc1qpq2cfgz5lktxzr5zqv7nrzz46hsvq3492ump9pz8rzcl8wqtwqcspx5y6a",
     )
@@ -54,14 +54,14 @@ fn main() {
         )
         .unwrap()
         .derive(bitcoin::util::bip32::ChildNumber::from_normal_idx(5).unwrap())
-        .address(bitcoin::Network::Bitcoin, desc_ctx).unwrap();
+        .address(desc_ctx, bitcoin::Network::Bitcoin).unwrap();
 
     let addr_two = Descriptor::<DescriptorPublicKey>::from_str(
             "sh(wsh(sortedmulti(1,xpub69H7F5d8KSRgmmdJg2KhpAK8SR3DjMwAdkxj3ZuxV27CprR9LgpeyGmXUbC6wb7ERfvrnKZjXoUmmDznezpbZb7ap6r1D3tgFxHmwMkQTPH/0/0/*,xpub661MyMwAqRbcFW31YEwpkMuc5THy2PSt5bDMsktWQcFF8syAmRUapSCGu8ED9W6oDMSgv6Zz8idoc4a6mr8BDzTJY47LJhkJ8UB7WEGuduB/1/0/*)))",
         )
         .unwrap()
         .derive(bitcoin::util::bip32::ChildNumber::from_normal_idx(5).unwrap())
-        .address(bitcoin::Network::Bitcoin, desc_ctx).unwrap();
+        .address(desc_ctx, bitcoin::Network::Bitcoin).unwrap();
     let expected = bitcoin::Address::from_str("325zcVBN5o2eqqqtGwPjmtDd8dJRyYP82s").unwrap();
     assert_eq!(addr_one, expected);
     assert_eq!(addr_two, expected);

src/descriptor/bare.rs

@@ -159,10 +159,7 @@ where
         Ok((witness, script_sig))
     }
 
-    fn max_satisfaction_weight<ToPkCtx: Copy>(&self) -> Option<usize>
-    where
-        Pk: ToPublicKey<ToPkCtx>,
-    {
+    fn max_satisfaction_weight(&self) -> Option<usize> {
         let scriptsig_len = self.ms.max_satisfaction_size()?;
         Some(4 * (varint_len(scriptsig_len) + scriptsig_len))
     }
@@ -340,10 +337,7 @@ where
         }
     }
 
-    fn max_satisfaction_weight<ToPkCtx: Copy>(&self) -> Option<usize>
-    where
-        Pk: ToPublicKey<ToPkCtx>,
-    {
+    fn max_satisfaction_weight(&self) -> Option<usize> {
         Some(4 * (1 + 73 + self.pk.serialized_len()))
     }