BlockChain sync and Client app #55
Conversation
@@ -272,7 +272,7 @@ impl BlockChain { | |||
// create views onto rlp | |||
let block = BlockView::new(bytes); | |||
let header = block.header_view(); | |||
let hash = block.sha3(); | |||
let hash = header.sha3(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
correct
Please check the client interface in src/client.rs, changes in header.rs and everything else :) |
for i in 13..d.len() { | ||
blockheader.seal.push(try!(Decodable::decode(&d[i]))); | ||
for i in 13..r.item_count() { | ||
blockheader.seal.push(try!(r.val_at(i))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is now corrected in master (should be raw RLP, not the string value).
@@ -104,3 +104,7 @@ pub mod genesis; | |||
pub mod views; | |||
pub mod blockchain; | |||
pub mod extras; | |||
pub mod client; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
client module is missing ;)
file not found for module `client`
src/lib.rs:107 pub mod client;
|
||
#[derive(PartialEq, Eq, Debug)] | ||
enum PeerAsking | ||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
putting {
s on newlines, now? :-)
All issues addressed |
pub fn clear(&mut self) { | ||
} | ||
|
||
pub fn import_block(&mut self, bytes: &[u8]) -> ImportResult { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docs?
@@ -113,7 +113,7 @@ impl Spec { | |||
let r = Rlp::new(&seal); | |||
(0..self.seal_fields).map(|i| r.at(i).as_raw().to_vec()).collect() | |||
}, | |||
hash: RefCell::new(None), | |||
hash: RefCell::new(None) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
always leave trailing comma, no?
BlockChain sync and Client app
json-spec: fix clique epoch to non-zero u64
No description provided.