tc39 / proposal-shadowrealm

ECMAScript Proposal, specs, and reference implementation for Realms
https://tc39.es/proposal-shadowrealm/
1.44k stars 67 forks source link

Normative: Wrapped function should reuse target name and length #339

Closed leobalter closed 2 years ago

leobalter commented 3 years ago

Closes #338

leobalter commented 3 years ago

@ljharb PTAL

leobalter commented 3 years ago

@ljharb we have the formatted header now. Is there anything else for this PR?

Jack-Works commented 3 years ago

What's the prior art of this change? If it's performance consideration, I think it should be host defined, host either copy the name and length or do nothings

mhofman commented 3 years ago

I'd be opposed to host defined behavior. We want more predictable behavior, not less

jdalton commented 3 years ago

I like the being similar to bind approach too.

leobalter commented 3 years ago

This PR should observe any outcome from https://github.com/tc39/proposal-shadowrealm/issues/338#issuecomment-964606972

leobalter commented 2 years ago

This achieved consensus during the TC39 plenary in Dec 2021.