Created attachment 357411 [details] [diff] [review] Patch SyntaxError is in ES3, but missing from Tamarin (because the need for it has been slight, to say the least). This patch reintroduces it. SyntaxError is required by the run-time compiler. The patch could be #ifdef'd by the presence of the run-time compiler, but it's a tiny amount of code and leaving it in brings us closer to ES3 in any case.
Attachment #357411 - Flags: review?(edwsmith)
Comment on attachment 357411 [details] [diff] [review] Patch i suppose we could have used this instead of VerifyError, but oh well, hindsight.
I think you'll also need to define SyntaxError in Error.as, no? anyway, whats in the patch looks fine.
(In reply to comment #2) > I think you'll also need to define SyntaxError in Error.as, no? anyway, whats > in the patch looks fine. Turns out it was already defined there! And the class is defined in core already, of course. I overstated the case for the fix. All it does is make SyntaxError available to C++ code.
redux changeset: 1311:e44005c69314
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
SyntaxError tested in acceptance/ecma3/ErrorObject/e15_11_1.as as well as various files in acceptance/as3/Statements/Exceptions/
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.