-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Fix #340 and DropAfterBlockBySha/NewestSha bug. #357
Conversation
@@ -0,0 +1,182 @@ | |||
// Copyright (c) 2013-2014 Conformal Systems LLC. |
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.
New file. This should be 2015.
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.
Whoops. Will fix that.
d6e5d56
to
055bfc2
Compare
@aakselrod Thanks for hunting this down and getting it fixed. Looks good. OK |
if rintbuf != uint32(network) { | ||
break | ||
} | ||
err = binary.Read(dr, binary.LittleEndian, &rintbuf) |
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 error isn't checked
Fix looks correct but I added some suggestions for the test code. |
Updated test code to comply with @jrick's suggestions. |
spentTxList = spentTxList[:len(spentTxList)-1] | ||
// XXX format sTxList and set update Table | ||
// This code should never be hit - aakselrod | ||
return fmt.Errorf( |
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.
Argument syntax formatting here doesn't match the rest of the code base.
- Delete spent TX in setclearSpentData when unspent by block disconnect on reorg; return an error when there's more than one record to delete in the spent TX as that should never happen. - Test spent TX deletion when reorg causes block disconnect. - Test for correct NewestSha results after DropAfterBlockBySha. - Fix DropAfterBlockBySha to update info for NewestSha. - Updated copyright statements in modified files
All set. Round 2 OK. |
ok |
disconnect on reorg.