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

More sync fixes #685

Merged
merged 5 commits into from Mar 13, 2016
Merged

More sync fixes #685

merged 5 commits into from Mar 13, 2016

Conversation

arkpar
Copy link
Collaborator

@arkpar arkpar commented Mar 11, 2016

Fixes #662

if needed_bodies.len() >= MAX_BODIES_TO_REQUEST {
break;
}
if let Some((start, ref items)) = self.headers.range_iter().next() {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Download bodies only for the validated part of the header-chain

@arkpar arkpar added A0-pleasereview 🤓 Pull request needs code review. A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. and removed A0-pleasereview 🤓 Pull request needs code review. A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Mar 11, 2016
@arkpar arkpar force-pushed the sync branch 2 times, most recently from 059af81 to ac1c249 Compare March 12, 2016 08:58
@arkpar arkpar added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Mar 13, 2016
gavofyork added a commit that referenced this pull request Mar 13, 2016
@gavofyork gavofyork merged commit 9493238 into master Mar 13, 2016
@gavofyork gavofyork deleted the sync branch March 13, 2016 11:14
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

2 participants