gh-95707: Fix function signature

[tiran]

• Issue: Support the use of the managed pre-header in builtin classes. #95707

[markshannon]

Thanks for fixing this.

It's nice to have runtime type checking, but the error message is not very informative.

RuntimeError: null function or function signature mismatch

Is it null or not? What function?
Would it be possible to improve the error message, so those of us less familiar with web assembly can fix them?

[markshannon]

@tiran Want to run the buildbots on this?
I'm happy to merge it, otherwise.

[markshannon]

I'll take that as a "merge" then 🙂

[tiran]

No need to run buildbots. I verified the patch locally.

It is a bit unfortunate that WASM runtimes and Emscripten do not provide more debug information. The error message is coming from NodeJS, probably from the V8 JavaScript engine itself. There is an outstanding feature request to improve the situation, emscripten-core/emscripten#16126 . -Wbad-function-cast -Wcast-function-type detects some mismatches, but it would not detect this mismatch.

@hoodmane has been looking into the problem, too.