Skip to content

Avoid re-encrypting chunks if file has not been modified.#160

Merged
2 commits merged intoWithAutonomi:masterfrom
Fraser999:prepare_window
Jan 27, 2016
Merged

Avoid re-encrypting chunks if file has not been modified.#160
2 commits merged intoWithAutonomi:masterfrom
Fraser999:prepare_window

Conversation

@Fraser999
Copy link
Copy Markdown
Contributor

Review on Reviewable

@maidsafe-highfive
Copy link
Copy Markdown

r? @brian-js

(maidsafe_highfive has picked a reviewer for you, use r? to override)

@ghost
Copy link
Copy Markdown

ghost commented Jan 27, 2016

fn close will have to be changed also, see #163.

ghost pushed a commit that referenced this pull request Jan 27, 2016
Avoid re-encrypting chunks if file has not been modified.
@ghost ghost merged commit fd3c292 into WithAutonomi:master Jan 27, 2016
@Fraser999 Fraser999 deleted the prepare_window branch January 27, 2016 14:22
@ghost
Copy link
Copy Markdown

ghost commented Jan 27, 2016

This will fail the test added by #163...

@ghost
Copy link
Copy Markdown

ghost commented Jan 27, 2016

@lgvz Shall I fix it in a separate PR, or would you be ok do it in #163?

@ghost
Copy link
Copy Markdown

ghost commented Jan 27, 2016

@brian-js whatever works for you.

@ghost
Copy link
Copy Markdown

ghost commented Jan 27, 2016

Ok, I'll push a fix that'll allow your new test to pass.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants