Skip to content

Reuse large lists #2331

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

Merged
merged 6 commits into from
Jun 26, 2025
Merged

Conversation

wixoaGit
Copy link
Member

@wixoaGit wixoaGit commented Jun 22, 2025

Keep a list to reuse later if it can hold at least 2048 a configurable amount of elements. Paradise was using block() to grab a large list of turfs thousands of times during init, allocating gigabytes. This optimizes that.

@boring-cyborg boring-cyborg bot added the Runtime Involves the OpenDream server/runtime label Jun 22, 2025
@boring-cyborg boring-cyborg bot added the Client Involves the OpenDream client label Jun 26, 2025
@github-actions github-actions bot added size/M and removed size/S labels Jun 26, 2025
Co-authored-by: Moony <moony@hellomouse.net>
@wixoaGit wixoaGit merged commit 9603390 into OpenDreamProject:master Jun 26, 2025
7 checks passed
@wixoaGit wixoaGit deleted the reuse_large_lists branch June 26, 2025 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Client Involves the OpenDream client Runtime Involves the OpenDream server/runtime size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants