Skip to content

Implements a handful of contents lists' ContainsValue methods#2587

Merged
wixoaGit merged 12 commits into
OpenDreamProject:masterfrom
tontyGH:contents-list-implementations
May 24, 2026
Merged

Implements a handful of contents lists' ContainsValue methods#2587
wixoaGit merged 12 commits into
OpenDreamProject:masterfrom
tontyGH:contents-list-implementations

Conversation

@tontyGH
Copy link
Copy Markdown
Contributor

@tontyGH tontyGH commented May 24, 2026

No description provided.

@boring-cyborg boring-cyborg Bot added the Runtime Involves the OpenDream server/runtime label May 24, 2026
@tontyGH tontyGH changed the title Implements a handful of contents lists' ContainsValue Implements a handful of contents lists' ContainsValue methods May 24, 2026
Comment thread OpenDreamRuntime/Objects/Types/DreamList.cs Fixed
Comment thread OpenDreamRuntime/Objects/Types/DreamList.cs Outdated
Comment thread OpenDreamRuntime/Objects/Types/DreamList.cs Outdated
@tontyGH tontyGH requested a review from wixoaGit May 24, 2026 23:17
Comment thread OpenDreamRuntime/Objects/Types/DreamList.cs Fixed
Comment thread OpenDreamRuntime/Objects/Types/DreamList.cs Fixed
Comment thread OpenDreamRuntime/Objects/Types/DreamList.cs Fixed
@tontyGH
Copy link
Copy Markdown
Contributor Author

tontyGH commented May 24, 2026

I got baited...

@wixoaGit
Copy link
Copy Markdown
Member

I HONESTLY SHOULD BE ALLOWED TO DO THAT THOUGH

You could, they just have to be lowercase if they're gonna be local variables.

@tontyGH
Copy link
Copy Markdown
Contributor Author

tontyGH commented May 24, 2026

Oh.

Comment thread OpenDreamRuntime/Objects/Types/DreamList.cs Outdated
@wixoaGit wixoaGit enabled auto-merge (squash) May 24, 2026 23:29
@wixoaGit wixoaGit disabled auto-merge May 24, 2026 23:31
tontyGH added 2 commits May 24, 2026 19:32
TurfContentsList.GetValue automatically handles ref counting for us
@wixoaGit wixoaGit enabled auto-merge (squash) May 24, 2026 23:34
@wixoaGit wixoaGit merged commit 537c100 into OpenDreamProject:master May 24, 2026
20 checks passed
@tontyGH tontyGH deleted the contents-list-implementations branch May 24, 2026 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Runtime Involves the OpenDream server/runtime size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants