Loader helpers could be namespaced

[JamesLMilner]

At the moment the loader helpers put themselves onto the returned module, with a lot of potential for naming clashes. It seems like it would be sensible to namespace them somehow rather them attaching them onto the returned module?

To understand what I'm talking about:

instantiateStreaming(fetch("build/optimized.wasm"))
    .then(wasmModule => { // Type: ASUtil
         // wasmModule.getString  - Loader helper
         // wasmModule.add - Our exported function

[dcodeIO]

Related, and most likely fixed within: #411

[JamesLMilner]

@dcodeIO did you end up reverting that in the end? Looking at it those functions would be namespaced to memory or table respectively.

[dcodeIO]

Yeah, had to revert it unfortunately. I am also fine with adding this to the existing loader, i.e. entirely avoiding low-level TypeScript hell, if someone is interested in doing so.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.