The MODULARIZE_INSTANCE option was originally meant to be a tiny little
convenient option, as described in settings.js:

// Similar to MODULARIZE, but while that mode exports a function, with
which you
// can create multiple instances, this option exports a singleton instance.
In
// other words, it's the same as if you used MODULARIZE and did EXPORT_NAME
=
// EXPORT_NAME() to create the instance manually.

However, over time it's actually gotten complex to maintain. Worse, we are
working on a major rethinking of the MODULARIZE option [1], in order to
handle Promises correctly, and MODULARIZE_INSTANCE is making things much
more complex there.

We'd therefore like to remove MODULARIZE_INSTANCE (see [2] for PR).
Removing it will allow us to fix the Promise problems with MODULARIZE.
After that is done, if there is interest we can think about how a new
MODULARIZE_INSTANCE would work on top of that.

For people using MODULARIZE_INSTANCE today, after we remove it you can do
what the settings.js comment says, that is, manually create the singleton
instance yourself. I realize that's not quite as convenient, but given the
above we (the people in the two PRs mentioned) think it's the best option
overall for the project.

Are there concerns with that, or things we've missed?

- Alon

[1] https://github.com/emscripten-core/emscripten/pull/10697
[2] https://github.com/emscripten-core/emscripten/pull/11037

-- 
You received this message because you are subscribed to the Google Groups 
"emscripten-discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/emscripten-discuss/CAEX4NpQLjGpRjEq43NpBZrP-4X-K4%2BY7cvwESyZ9n0yoU%3DgKHg%40mail.gmail.com.

Reply via email to