[WIP] Asset contracts 2.0

Cargo.toml

@@ -9,9 +9,9 @@ repository = "https://github.com/ElementsProject/rust-elements/"
 documentation = "https://docs.rs/elements/"
 
 [features]
-default = [ "json-contract" ]
+default = [ "contracts" ]
 
-json-contract = [ "serde_json" ]
+contracts = [ "serde", "serde_cbor", "serde_json" ]
 "serde-feature" = [
     "bitcoin/use-serde",
     "serde"
@@ -25,13 +25,13 @@ bitcoin = "0.23"
 # to avoid requiring two version of bitcoin_hashes.
 bitcoin_hashes = "0.7.6"
 
-# Used for ContractHash::from_json_contract.
-serde_json = { version = "<=1.0.44", optional = true }
+serde = { version = "1.0", optional = true, features = ["derive"] }
 
-[dependencies.serde]
-version = "1.0"
-optional = true
+# Used for contracts module.
+serde_cbor = { version = "0.11.1", optional = true }
+serde_json = { version = "<=1.0.44", optional = true }
 
 [dev-dependencies]
 rand = "0.6.5"
 serde_json = "<=1.0.44"
+bitcoin = { version = "0.23", features = ["use-serde"] }

src/address.rs

@@ -30,7 +30,7 @@ use bitcoin::util::base58;
 use bitcoin::PublicKey;
 use bitcoin::hashes::Hash;
 use bitcoin::secp256k1;
-#[cfg(feature = "serde")]
+#[cfg(feature = "serde-feature")]
 use serde;
 
 use blech32;
@@ -75,6 +75,7 @@ impl fmt::Display for AddressError {
     }
 }
 
+#[allow(deprecated)]
 impl error::Error for AddressError {
     fn cause(&self) -> Option<&error::Error> {
         match *self {
@@ -87,16 +88,7 @@ impl error::Error for AddressError {
     }
 
     fn description(&self) -> &str {
-        match *self {
-            AddressError::Base58(ref e) => e.description(),
-            AddressError::Bech32(ref e) => e.description(),
-            AddressError::Blech32(ref e) => e.description(),
-            AddressError::InvalidAddress(..) => "was unable to parse the address",
-            AddressError::UnsupportedWitnessVersion(..) => "unsupported witness version",
-            AddressError::InvalidBlindingPubKey(..) => "an invalid blinding pubkey was encountered",
-            AddressError::InvalidWitnessProgramLength => "program length incompatible with version",
-            AddressError::InvalidWitnessVersion => "invalid witness script version",
-        }
+        "description() is deprecated; use Display"
     }
 }
 
@@ -603,7 +595,7 @@ impl FromStr for Address {
     }
 }
 
-#[cfg(feature = "serde")]
+#[cfg(feature = "serde-feature")]
 impl<'de> serde::Deserialize<'de> for Address {
     #[inline]
     fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
@@ -646,7 +638,7 @@ impl<'de> serde::Deserialize<'de> for Address {
     }
 }
 
-#[cfg(feature = "serde")]
+#[cfg(feature = "serde-feature")]
 impl serde::Serialize for Address {
     fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
     where
@@ -662,8 +654,6 @@ mod test {
     use bitcoin::util::key;
     use bitcoin::Script;
     use bitcoin::secp256k1::{PublicKey, Secp256k1};
-    #[cfg(feature = "serde")]
-    use serde_json;
 
     fn roundtrips(addr: &Address) {
         assert_eq!(
@@ -678,9 +668,9 @@ mod test {
             "script round-trip failed for {}",
             addr,
         );
-        #[cfg(feature = "serde")]
+        #[cfg(feature = "serde-feature")]
         assert_eq!(
-            serde_json::from_value::<Address>(serde_json::to_value(&addr).unwrap()).ok().as_ref(),
+            ::serde_json::from_value::<Address>(serde_json::to_value(&addr).unwrap()).ok().as_ref(),
             Some(addr)
         );
     }

src/block.rs

@@ -21,8 +21,8 @@ use bitcoin;
 use bitcoin::blockdata::script::Script;
 use bitcoin::{BitcoinHash, BlockHash, VarInt};
 use bitcoin::hashes::{Hash, sha256};
-#[cfg(feature = "serde")] use serde::{Deserialize, Deserializer, Serialize, Serializer};
-#[cfg(feature = "serde")] use std::fmt;
+#[cfg(feature = "serde-feature")] use serde::{Deserialize, Deserializer, Serialize, Serializer};
+#[cfg(feature = "serde-feature")] use std::fmt;
 
 use dynafed;
 use Transaction;
@@ -49,7 +49,7 @@ pub enum ExtData {
     },
 }
 
-#[cfg(feature = "serde")]
+#[cfg(feature = "serde-feature")]
 impl<'de> Deserialize<'de> for ExtData {
     fn deserialize<D: Deserializer<'de>>(d: D) -> Result<Self, D::Error> {
         use serde::de;
@@ -149,7 +149,7 @@ impl<'de> Deserialize<'de> for ExtData {
     }
 }
 
-#[cfg(feature = "serde")]
+#[cfg(feature = "serde-feature")]
 impl Serialize for ExtData {
     fn serialize<S: Serializer>(&self, s: S) -> Result<S::Ok, S::Error> {
         use serde::ser::SerializeStruct;

src/confidential.rs

@@ -17,7 +17,7 @@
 //! Structures representing Pedersen commitments of various types
 //!
 
-#[cfg(feature = "serde")]
+#[cfg(feature = "serde-feature")]
 use serde::{Deserialize, Deserializer, Serialize, Serializer};
 
 use std::{io, fmt};
@@ -137,7 +137,7 @@ macro_rules! impl_confidential_commitment {
             }
         }
 
-        #[cfg(feature = "serde")]
+        #[cfg(feature = "serde-feature")]
         impl Serialize for $name {
             fn serialize<S: Serializer>(&self, s: S) -> Result<S::Ok, S::Error> {
                 use serde::ser::SerializeSeq;
@@ -161,7 +161,7 @@ macro_rules! impl_confidential_commitment {
             }
         }
 
-        #[cfg(feature = "serde")]
+        #[cfg(feature = "serde-feature")]
         impl<'de> Deserialize<'de> for $name {
             fn deserialize<D: Deserializer<'de>>(d: D) -> Result<Self, D::Error> {
                 use serde::de::{Error, Visitor, SeqAccess};