Jump to content

Pull Request: CInterface Update


LukasPJ

Recommended Posts

Pull request: https://github.com/GarageGames/Torque3D/pull/2136

Branch: https://github.com/lukaspj/Torque3D/tree/cinterface-update


The PR seeks to update the CInterface, which was left in order to integrate JavaScript with T3D.


My main issues with the existing solution was that it still used the TorqueScript console for method calls, which is inefficient compared to calling functions directly. Also it didn't really work any more.

Furthermore, this CInterface seeks to transfer the control of method-lookups and the like from the console to the script that interfaces with the engine.


Main issues that needs to be addressed:


Script->Cinterface calls

In the codeInterpreter.cpp, TorqueScript calls the CInterface, this code should be reviewed / verified:

https://github.com/GarageGames/Torque3D/pull/2136/files#diff-5ec52f3c77bc47c45a6b93b21c6f783b


In console.cpp, the CInterface is called before the console is inquired about evaluation of the function call. This should be reviewed / verified:

https://github.com/GarageGames/Torque3D/pull/2136/files#diff-03ce4e39e998227a38fd95067262f6ad



Cinterface->Engine calls

In engineTypes.h, the ValueType of EngineStructTypeTraits has been changed, despite the fact there was a clear reason that it had the original type to begin with. This should be reviewed / verified:

https://github.com/GarageGames/Torque3D/pull/2136/files#diff-da7328c8cb002a64b343c49c865a5cc6

Link to comment
Share on other sites

  • 5 years later...
  • 2 weeks later...
  • 1 month later...
  • 1 month later...

I missed this it seems 🙂

But if you just want C and T3D then no reason to do any fancy stuff, you can just call functions directly from C.
The biggest issue for you would be going the other way, but that should be easy to overcome let me know if you need any help, I'm pretty hung up with kids and work though so I'm easier to reach through Discord. 

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...