Skip to content

clear_ref_userdata is variadic#5

Merged
Y0SH1M4S73R merged 4 commits into
masterfrom
variadic_clear_ref_userdata
Jun 3, 2026
Merged

clear_ref_userdata is variadic#5
Y0SH1M4S73R merged 4 commits into
masterfrom
variadic_clear_ref_userdata

Conversation

@Y0SH1M4S73R

Copy link
Copy Markdown
Collaborator

Updates meowtonin to 0.3.1, allowing byond_fn functions to be variadic.
Updates mlua because the version currently being used had a feature that would be rejected in future rust versions.
Makes clear_ref_userdata varidic to reduce FFI overhead when clearing userdata for multiple objects.

@Y0SH1M4S73R

Copy link
Copy Markdown
Collaborator Author

I was thinking of testing this locally before merging due to CI not being able to pull up a runner, but I'm afraid this PR's gonna be blocked until there is a BYOND version post-1674 that isn't subtly borked in a way that causes client crashes that can cause restart-necessitating OS-level issues.

@SyncIt21

SyncIt21 commented May 27, 2026

Copy link
Copy Markdown

CC @Y0SH1M4S73R

Latest version is 516.1681

@Y0SH1M4S73R

Copy link
Copy Markdown
Collaborator Author

You know what I'm not gonna wait for this to build I tested it and all the tests passed.

@Y0SH1M4S73R Y0SH1M4S73R merged commit e43673d into master Jun 3, 2026
1 check failed
@SyncIt21

SyncIt21 commented Jun 3, 2026

Copy link
Copy Markdown

Thanks for updating the repo. Can you make a pull updating dreamlua in the main repo? The debugger doesn't work on the latest byond version

@Y0SH1M4S73R

Copy link
Copy Markdown
Collaborator Author

Thanks for updating the repo. Can you make a pull updating dreamlua in the main repo? The debugger doesn't work on later versions

Only if the live servers are running on 1674 or later. The updated meowtonin dependency depends on breaking changes to byondapi made in 1674.

@SyncIt21

SyncIt21 commented Jun 3, 2026

Copy link
Copy Markdown

Got it. You should probably ping oranges on discord to update the servers. I'll try to get in touch with him as well

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.

2 participants