-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
OpenDream fast del() support #83237
OpenDream fast del() support #83237
Conversation
this wont work because you change the define entirely which means switch statements / if statements will perform logic wrong |
Can confirm OpenDream spends more time queuing things in SSgarbage than it would take to just harddel them. |
f3e1ef1
to
81c2ebd
Compare
Also there's no reason outside of unit test times to speed this up queue size impacts very little |
Not safe as existing code expects a vague valid reference window and this voids that contract |
I fixed that by using |
#ifdef OPENDREAM
if(hint != QDEL_HINT_IWILLGC || hint != QDEL_HINT_LETMELIVE)
hint = QDEL_HINT_HARDDEL
#endif just fucking use this in /proc/qdel |
f7d3c62
to
99739a7
Compare
bikeshed |
About The Pull Request
Skips GC on OpenDream
Why It's Good For The Game
OpenDream's implementation makes del() super fast and not to be feared. Save a ton of bytecode processing by ridding the queue logic.
Changelog
Not really.