Merge pull request #40 from gavofyork/rlp_array

missing rustc_serialize crate && rlp `as_list` function

Cargo.toml

@@ -11,6 +11,7 @@ log = "0.3"
 env_logger = "0.3"
 ethcore-util = "0.1.0"
 evmjit = { path = "rust-evmjit", optional = true }
+rustc-serialize = "0.3"
 
 [features]
 jit = ["evmjit"]

src/account.rs

@@ -169,6 +169,8 @@ fn playpen() {
 
 #[test]
 fn new_account() {
+	use rustc_serialize::hex::ToHex;
+
 	let a = Account::new(U256::from(69u8), U256::from(0u8), HashMap::new(), Bytes::new());
 	assert_eq!(a.rlp().to_hex(), "f8448045a056e81f171bcc55a6ff8345e692c0f86e5b48e01b996cadc001622fb5e363b421a0c5d2460186f7233c927e7db2dcc703c0e500b653ca82273b7bfad8045d85a470");
 	assert_eq!(a.balance(), &U256::from(69u8));
@@ -179,6 +181,8 @@ fn new_account() {
 
 #[test]
 fn create_account() {
+	use rustc_serialize::hex::ToHex;
+
 	let a = Account::new(U256::from(69u8), U256::from(0u8), HashMap::new(), Bytes::new());
 	assert_eq!(a.rlp().to_hex(), "f8448045a056e81f171bcc55a6ff8345e692c0f86e5b48e01b996cadc001622fb5e363b421a0c5d2460186f7233c927e7db2dcc703c0e500b653ca82273b7bfad8045d85a470");
 }

src/blockheader.rs

@@ -56,7 +56,8 @@ impl Header {
 
 impl Decodable for Header {
 	fn decode<D>(decoder: &D) -> Result<Self, DecoderError> where D: Decoder {
-		decoder.read_list(| d | {
+		let d = try!(decoder.as_list());
+
 		let blockheader = Header {
 			parent_hash: try!(Decodable::decode(&d[0])),
 			uncles_hash: try!(Decodable::decode(&d[1])),
@@ -75,7 +76,6 @@ impl Decodable for Header {
 		};
 		// TODO: fill blockheader.seal with (raw) list items index 12..)
 		Ok(blockheader)
-		})
 	}
 }
 

src/lib.rs

@@ -72,6 +72,7 @@
 #[macro_use]
 extern crate log;
 extern crate env_logger;
+extern crate rustc_serialize;
 #[cfg(feature = "jit" )]
 extern crate evmjit;
 extern crate ethcore_util as util;

src/transaction.rs

@@ -37,7 +37,8 @@ impl Encodable for Transaction {
 
 impl Decodable for Transaction {
 	fn decode<D>(decoder: &D) -> Result<Self, DecoderError>  where D: Decoder {
-		decoder.read_list(| d | {
+		let d = try!(decoder.as_list());
+
 		let transaction = Transaction {
 			nonce: try!(Decodable::decode(&d[0])),
 			gas_price: try!(Decodable::decode(&d[1])),
@@ -46,8 +47,8 @@ impl Decodable for Transaction {
 			value: try!(Decodable::decode(&d[4])),
 			data: try!(Decodable::decode(&d[5])),
 		};
+
 		Ok(transaction)
-		})
 	}
 }