On 2015/06/19 11:38:25, rossberg wrote:
On 2015/06/19 09:43:54, Jakob wrote:
> Yeah, node.js developers will love this.
>
> [+rossberg] Andreas, do the import/export statements (look at
codemap.js for
an
> example) conflict with official ECMAScript module plans? Once we enable
this
> support, people will get mad if we break it again, even temporarily.
I don't see any compatibility problems.
Yet, I'm not fond of putting embedder-specific hacks into the V8
implementation
itself. In particular, since it's not possible to have regression tests
for
that
on our side. Isn't there a way to address this on the node side? E.g., by
having
a simple build script that wraps the respective files with the same
statements
that this CL adds?
It looks like modifying the node build process to wrap these files would be
difficult. Would it be preferable to use ES6 modules syntax instead of node
specific constructs to implement this change inside V8?
https://codereview.chromium.org/1179173009/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
