-
Notifications
You must be signed in to change notification settings - Fork 15
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
Removed SF table clutter in favor of using type metatables. #371
base: develop
Are you sure you want to change the base?
Conversation
You missed these: Also in player_metamethods.__wrap = wrap
player_metamethods.__unwrap = unwrap but they are already set when doing SF.AddObject(Un)Wrapper. So they aren't needed. Not sure if this is done anywhere else though. Really good job though btw, lets just hope you didn't accidentally mix up some of the names xD |
Yeah, will need to be careful about these. Lots of testing. Was thinking about this change only 3 days ago :) good job |
It also appears to break the ACF-Starfall implementation which kinda sucks but I guess the guys at ACF will just have to do an update. Edit: Nevermind, looks like you're a step ahead. |
Will also break any extension addons relying on existing libraries. |
Didn't realize SF.AddObject(Un)Wrapper set the __(un)wrap key. Oops. All other mentioned instances have been fixed. I'll be on the iG server testing every func if anybody wants to come double-team it. |
Whilst testing, it might be useful to develop SF scripts as unit-tests for library functions. So if you're doing the work already, might aswell save and post the SF chips "somewhere". Just a thought. Reproducible tests are good. |
Forgot about holograms lib. Of course I remember as I am about to leave. Will fix when I return. |
Ehh, did it right now and checked all files with regex, did not catch anything with this.
|
No description provided.