Skip to content

Re-implement range() and orange() to match BYOND#2604

Open
tontyGH wants to merge 3 commits into
OpenDreamProject:masterfrom
tontyGH:range-refactor
Open

Re-implement range() and orange() to match BYOND#2604
tontyGH wants to merge 3 commits into
OpenDreamProject:masterfrom
tontyGH:range-refactor

Conversation

@tontyGH
Copy link
Copy Markdown
Contributor

@tontyGH tontyGH commented May 28, 2026

Fixes #2601
Fixes #2584

Unless there's some niche interaction I'm unaware of the output of this implementation should match BYOND 1-to-1
I may have overengineered the integration test

@boring-cyborg boring-cyborg Bot added Client Involves the OpenDream client Runtime Involves the OpenDream server/runtime labels May 28, 2026
Comment thread OpenDreamRuntime/Procs/Native/DreamProcNativeHelpers.cs Fixed
Comment thread OpenDreamRuntime/Procs/Native/DreamProcNativeHelpers.cs Fixed
Comment thread OpenDreamRuntime/Procs/Native/DreamProcNativeHelpers.cs Fixed
Comment thread OpenDreamRuntime/Procs/Native/DreamProcNativeHelpers.cs Fixed
@ZeWaka
Copy link
Copy Markdown
Collaborator

ZeWaka commented May 29, 2026

orange!!!!! 🍊

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/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

range() does not include found areas Any proc that uses default view range throws exception if world.view is not 1:1

3 participants