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

[Heavily WIP] SQRat #732

Closed
wants to merge 39 commits into from
Closed

[Heavily WIP] SQRat #732

wants to merge 39 commits into from

Conversation

tobbi
Copy link
Member

@tobbi tobbi commented Aug 24, 2017

This is, as the title says, heavily work in progress. Currently, the following works:

Calling functions of objects that have them exposed (currently: AmbientSound and Player).

For every object there is, we create a class type sqratclass_ + and an instance type: sqratinstance_ + . I know this is totally stupid, since we should create just one class type and then instances of it when the objects are created actually, but who cares. I'll fix it later, probably.

So, for example for the menu level, you can do:
sqratinstance_Tux.deactivate();

  • Get sqratthread to work with internal scheduler
  • Don't pass VM to each function (use Sqrat::DefaultVM() instead)
  • Expose color constants to squirrel

@maxteufel maxteufel added category:code involves:scripting status:in-progress Progress has been done, but more is intended be done status:needs-review Work needs to be reviewed by other people status:needs-work In progress, but no one is currently working on it (New volunteers welcome) type:feature labels Aug 25, 2017
@tobbi
Copy link
Member Author

tobbi commented Aug 25, 2017

The Travis build fails because of warnings in SQRat code, not ours. phew

@maxteufel
Copy link
Member

Reference: #650 ("Use SQRat")

@tobbi tobbi force-pushed the sqrat branch 2 times, most recently from ec8dd34 to 03ec2c7 Compare December 3, 2017 02:23
For some reason, some functions are exposed to more than one table.
@tobbi
Copy link
Member Author

tobbi commented Dec 17, 2017

Threading is bound to give us some problems. If anyone could implement the threading scripting functions, feel free. I'm struggling with them.

@tobbi
Copy link
Member Author

tobbi commented Nov 15, 2019

This issue was abandoned.

@tobbi tobbi closed this Nov 15, 2019
@mrkubax10 mrkubax10 removed the status:needs-review Work needs to be reviewed by other people label Jul 20, 2023
@tobbi tobbi mentioned this pull request Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:code involves:scripting status:in-progress Progress has been done, but more is intended be done status:needs-work In progress, but no one is currently working on it (New volunteers welcome) type:feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants