search

audreyt / node-webworker-threads

Lightweight Web Worker API implementation with native threads
https://npmjs.org/package/webworker-threads
Other
2.3k stars 149 forks source link

Extended support for importScripts #7

Open jolcese opened 11 years ago

jolcese commented 11 years ago

importScripts currently only supports laoding a file from disk. On the W3C standard (http://www.w3.org/TR/workers/#importing-scripts-and-libraries), it support http:// & https:// urls Would be great if supported

audreyt commented 11 years ago

Thanks for the suggestion! Synchronous fetch entails an additional dependency on e.g. libcurl. While certainly feasible, I'd like to make it optional (i.e. only enabled when libcurl is available on the system) as opposed to mandatory (i.e. build libcurl as part of this module). Would that make sense to you?

jolcese commented 11 years ago

Absolutely! I agree that it should be optional and not mandatory. You rock!