Closed Bug 555104 Opened 10 years ago Closed 10 years ago

remove JS_ExecuteScriptPart

Categories

(Core :: JavaScript Engine, defect)

x86
Linux
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: luke, Assigned: luke)

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 file)

Attached patch do itSplinter Review
It seems like there are no uses; less is more.
Attachment #435052 - Flags: review?(brendan)
Comment on attachment 435052 [details] [diff] [review]
do it

To quote Arnie, "Hasta la vista, baby".

This was misbegotten from day one. True story: the very first JS impl ("Mocha") would bind global functions (the only kind in Netscape 2) and vars at compile time. Moving all name binding to runtime via the prolog left some old embeddings wanting the compile-binds behavior, so I threw them this bone. I should have thrown a rock.

/be
Attachment #435052 - Flags: review?(brendan) → review+
http://hg.mozilla.org/tracemonkey/rev/e68d2b012d59
Whiteboard: fixed-in-tracemonkey
http://hg.mozilla.org/mozilla-central/rev/e68d2b012d59
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.