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
bip157: Fix typo on getcfcheckpt command string #367
Conversation
I can add tests if you show me how to get bytes vector samples for network messages |
Codecov Report
@@ Coverage Diff @@
## master #367 +/- ##
==========================================
- Coverage 82.56% 81.91% -0.65%
==========================================
Files 39 39
Lines 8236 7179 -1057
==========================================
- Hits 6800 5881 -919
+ Misses 1436 1298 -138
Continue to review full report at Codecov.
|
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.
LGTM. Thanks!
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.
utACK modulo testing that full round fails before this and succeed after.
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.
tACK. and checked against bitcoin/bitcoin#16442
…ents 0e0fa06 Simplify `Display` impl of `SerializedSignature` (Martin Habovstiak) 5d51b9d Added `MAX_LEN` constant to `serialized_signature` (Martin Habovstiak) e642a52 Add `#[inline]` to methods of `SerializedSignatre` (Martin Habovstiak) e92540b `impl IntoIterator for SerializedSignature` (Martin Habovstiak) 7f2d3d2 Move `SerializedSignature` into its own module (Martin Habovstiak) 901d5ff `impl<'a> IntoIterator for &'a SerializedSignature` (Martin Habovstiak) 1d2a1c3 Deduplicate `self.data[..self.len]` expressions (Martin Habovstiak) Pull request description: This * Deduplicates slicing operations * Implements `IntoIterator` (owned and borrowed) * Reorganizes the code for better clarity * Adds `#[inline]`s * Checks length set by libsep256k1 Closes rust-bitcoin#367 Closes rust-bitcoin#368 Individual commits are hopefully easier to review. ACKs for top commit: apoelstra: ACK 0e0fa06 Tree-SHA512: bbc759af767c8b84bfd6720456efc1e86da501aa193641dae3c99847a3c882f7d4aa7e5cbec074fdd9c2595f1f65e5fbb4c80620539a6357927149e5c2fbc734
This corrects a small typo on BIP157
getcfcheckpt
message command serialization/deserialization.cf: https://github.com/bitcoin/bips/blob/master/bip-0157.mediawiki#getcfcheckpt