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

Various of changes and fixes #10

Merged
merged 4 commits into from Sep 24, 2020
Merged

Conversation

TrueBrain
Copy link
Member

No description provided.

Otherwise it will start creating temporary files when files over
100k are uploaded. This has various of consequences, most noticeable:
the fds are not closed. This is cleaned up by the GC after a while,
but does increase the fd-count, and leaves stuff behind on the
disk.
On the OpenTTD language set, this reduces the memory usage from
505MB to 340MB. The only price we pay is that you cannot set
any random other field in the class, which is a good thing for
these kind of classes.
@LordAro LordAro merged commit df65125 into OpenTTD:master Sep 24, 2020
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

3 participants