-
Notifications
You must be signed in to change notification settings - Fork 215
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
initial support for bip30 #199
Conversation
@@ -68,6 +68,16 @@ impl ChainVerifier { | |||
|
|||
let coinbase_spends = block.transactions()[0].total_spends(); | |||
|
|||
// bip30 | |||
// TODO: add exceptions |
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.
we need to add exception right away, or it will fail on mainnet?
@@ -84,6 +84,8 @@ pub enum TransactionError { | |||
SigopsP2SH(usize), | |||
/// Coinbase transaction is found at position that is not 0 | |||
MisplacedCoinbase(usize), | |||
/// Not fully spent transaction with the same hash already exists. | |||
BIP30, |
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.
Will prefer actual reason of fail instead of bip30 reference which is quite large, but it is just me
fixed tests, syncing is now stuck at b73 |
@@ -28,6 +29,11 @@ impl ConsensusParams { | |||
}, | |||
} | |||
} | |||
|
|||
pub fn is_bip30_expcetion(&self, hash: &H256, height: u32) -> bool { |
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.
typo here
changes in
db/src/storage.rs
are only idiomatic