/utils

Track size and hash even when there is an error. #188

Merged
merged 5 commits into from Jan 20, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@ericsnowcurrently @jujubot
@ericsnowcurrently
Contributor

ericsnowcurrently commented Jan 19, 2016

Some readers/writers handle bytes in the same operation that they handle EOF. This change accommodates them.

(Review request: http://reviews.vapour.ws/r/3572/)
@ericsnowcurrently
Contributor

ericsnowcurrently commented Jan 20, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Jan 20, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju-utils

jujubot added a commit that referenced this pull request Jan 20, 2016

@jujubot
Merge pull request #188 from ericsnowcurrently/fix-file-wrappers-erro… 
…r-case

Track size and hash even when there is an error.

Some readers/writers handle bytes in the same operation that they handle EOF.  This change accommodates them.

(Review request: http://reviews.vapour.ws/r/3572/)
7195a14

@jujubot jujubot merged commit 7195a14 into juju:master Jan 20, 2016

@ericsnowcurrently ericsnowcurrently deleted the ericsnowcurrently:fix-file-wrappers-error-case branch Jan 20, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment