Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Rlp #207

Merged
merged 8 commits into from Jan 26, 2016
Merged

Rlp #207

merged 8 commits into from Jan 26, 2016

Conversation

NikVolf
Copy link
Contributor

@NikVolf NikVolf commented Jan 25, 2016

No description provided.

@arkpar arkpar added the A0-pleasereview 🤓 Pull request needs code review. label Jan 25, 2016
@@ -331,18 +331,31 @@ impl<'a> Decoder for BasicDecoder<'a> {
Some(l @ 0...0x7f) => Ok(try!(f(&[l]))),
// 0-55 bytes
Some(l @ 0x80...0xb7) => {
let d = &bytes[1..(1 + l as usize - 0x80)];
let last_index_of = 1 + l as usize - 0x80;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indention here is incorrect. Otherwise it looks good.

@debris debris added A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jan 26, 2016
@NikVolf NikVolf added the A0-pleasereview 🤓 Pull request needs code review. label Jan 26, 2016
@arkpar arkpar removed the A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. label Jan 26, 2016
debris added a commit that referenced this pull request Jan 26, 2016
@debris debris merged commit d30114b into master Jan 26, 2016
@debris debris deleted the rlp branch January 27, 2016 16:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-pleasereview 🤓 Pull request needs code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants