Hi,

The interface of the engine makes it possible to manually bind imported functions to native functions (instead of scripted functions from a module), which is very convenient if you want to create pseudo modules using native code: you can simply call BindImportedFunction() with the native function you want to bind.

However, the engine currently assumes that bound functions are always script functions (so doing what I described above crashes). Is there any good reason for that?

The attached patch seems to fix this (that's an ugly copy/paste from function call code so it can probably be cleaner than that). Would it make sense to support this behavior?

Btw, the m_regs.programPointer++; in the patch is a dumb copy/paste mistake...

The current import function feature is something I'm planning on retiring eventually.

woops, good to know! What is the reason for that? It sounded like a nice feature to have such a built-in linker!

The function handle can then be set from the application to point to whatever asIScriptFunction you want (as long as the signature matches ).

That's a nice idea, indeed! (with probably less overhead too). However, the syntax is not very nice if there are lots of functions with different types to import.

In fact my goal is to allow script users to dynamically load native symbols from the script before using them, with something like:

#include "ModuleAHeader" // define functions for module A
#include "ModuleBHeader" // define functions for module B

void main()
{
   // load the module (all native functions)
   LoadModule("ModuleA");

   // Use functions from module A
   FuncFromModuleA();
}

And in "ModuleA.h"

// simple function declaration, but which clearly states that it is imported
import void FuncFromModuleA() from "ModuleA";
 

This is to avoid binding all symbols at script startup (and maybe choose different implementations of the same symbols as well). Is there maybe another clean way to do this already?

Thanks! This also requires a change to the JIT. I'll submit a patch there too.

Your new system looks much better anyway. Do you know if this may happen in the near future?