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

Implements world.address & friends, adds remaining opendream_unimplemented world vars/procs #774

Merged
merged 3 commits into from
Oct 26, 2022

Conversation

Altoids1
Copy link
Contributor

@Altoids1 Altoids1 commented Aug 19, 2022

image

Summary

world.address and world.port are implemented, after a regression during RobustToolbox porting last year caused them to become commented out.

Makes #735 at least compile by setting up the associated opendream_unimplemented. Also went ahead and added every other one I could find in the ref; yell if I missed any.

Fixes #804.

Changelog

  • world.address and world.port have been added back in.
  • Several world vars and procs now exist as unimplemented.

@ike709
Copy link
Collaborator

ike709 commented Aug 19, 2022

(yet erroneously not marked as unimplemented).

Fairly certain the engine switch predates opendream_unimplemented

@github-actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

world.address
world.port
world.url
world.internet_address

EDIT: Does some various tweaking as per Wixoa's review
This should improve compile-ability, especially movement_mode.

EDIT: MOVEMENT_MODE macros now start at 0 instead of 1, as per parity
EDIT: Forgot to add cache_lifespan, lol
EDIT: Removing cache_lifespan since it now exists on master apparently
EDIT: also does Wixoa's new review
@wixoaGit wixoaGit merged commit 6e77bf9 into OpenDreamProject:master Oct 26, 2022
@Altoids1 Altoids1 deleted the around-the-world branch October 26, 2022 21:12
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.

Error setting vars on world
3 participants