Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix unclosed file descriptor after read_from_cache #65

Merged
merged 1 commit into from Sep 7, 2020

Conversation

korun
Copy link
Contributor

@korun korun commented Sep 3, 2020

Subj, IO.read ensures the file is closed before returning, while File.open(...).read leave unclosed descriptor.

Copy link
Owner

@spk spk left a comment

Choose a reason for hiding this comment

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

You're right nice catch thanks!

@spk spk merged commit 4dc5ab9 into spk:master Sep 7, 2020
@korun korun deleted the ensure-file-closed branch September 11, 2020 10:50
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.

None yet

2 participants