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

Removed SF table clutter in favor of using type metatables. #371

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Removed SF table clutter in favor of using type metatables. #371

wants to merge 1 commit into from

Conversation

davidsonbr
Copy link

No description provided.

@Jazzelhawk
Copy link
Member

@awilliamson
Copy link
Member

Yeah, will need to be careful about these. Lots of testing. Was thinking about this change only 3 days ago :) good job

@Jazzelhawk
Copy link
Member

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.

@awilliamson
Copy link
Member

Will also break any extension addons relying on existing libraries.

@davidsonbr
Copy link
Author

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.

@awilliamson
Copy link
Member

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.

@davidsonbr
Copy link
Author

Forgot about holograms lib. Of course I remember as I am about to leave. Will fix when I return.

@davidsonbr
Copy link
Author

Ehh, did it right now and checked all files with regex, did not catch anything with this.

SF.Entities|SF.Angles|SF.Color|SF.Players|SF.SGlobalTables|SF.Sounds|SF.Vectors|SF.Vehicles|SF.VMatrix|SF.Wire|SF.Holograms

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.

4 participants