Skip to content

Add flush(::Writer) to write central directory record without closing file#60

Merged
fhs merged 2 commits intofhs:masterfrom
tkf:flush
Nov 18, 2019
Merged

Add flush(::Writer) to write central directory record without closing file#60
fhs merged 2 commits intofhs:masterfrom
tkf:flush

Conversation

@tkf
Copy link
Contributor

@tkf tkf commented Nov 16, 2019

This PR implements flush(::Writer) to write out central directory record without closing the file. It is useful for, e.g., appending to a zip file in a long-running Julia process while keeping the zip file valid and readable by external programs.

Copy link
Owner

@fhs fhs left a comment

Choose a reason for hiding this comment

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

Thanks!

@tkf
Copy link
Contributor Author

tkf commented Nov 17, 2019

Thanks for the review! I added a few commens.

Copy link
Owner

@fhs fhs left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks!

@fhs fhs merged commit eeffee7 into fhs:master Nov 18, 2019
@tkf tkf deleted the flush branch November 18, 2019 19:17
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