Made RepositoryItemStream.Close idempotent so that it can be closed mult... #554

Merged
merged 1 commit into from Apr 6, 2013

Conversation

Projects
None yet
3 participants
Contributor

HenrikFrystykNielsen commented Apr 5, 2013

...iple times but only doing the cleanup on the first time through.

Avoided double-releasing the file lock on invalid range requests. Now we only release it once.

Henrik

Made RepositoryItemStream.Close idempotent so that it can be closed m…
…ultiple times but only doing the cleanup on the first time through.

Avoided double-releasing the file lock on invalid range requests. Now we only release it once.

Henrik
Owner

suwatch commented Apr 6, 2013

One way to know for certain is undoing this 6196157 and see.

@davidebbo davidebbo merged commit d604ecd into projectkudu:master Apr 6, 2013

Owner

davidebbo commented Apr 6, 2013

@suwatch yes, we did that, and it works fine.

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