Closed
Bug 32674
Opened 25 years ago
Closed 25 years ago
JS_SetVersion hides error; also, jsscan.c keyword bug
Categories
(Core :: JavaScript Engine, defect, P3)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
M15
People
(Reporter: brendan, Assigned: rogerl)
References
()
Details
(Keywords: js1.5)
Attachments
(1 file)
3.65 KB,
patch
|
Details | Diff | Splinter Review |
The patch in this bug fixes a performance and infallible-API-with-fallible-impl bug in JS_SetVersion. It also fixes what looks to me like a bug in the recent Unicode ECMA Edition 3 changes: if you ever spelled an identifier with Unicode escape sequence(s), you'd reset the unescaped atom's kwindex to -1, preventing even the unescaped spelling from being scanned as a reserved identifier. Roger, mccabe, can you code buddy? I'll check in if given the OK. /be
Reporter | ||
Updated•25 years ago
|
Assignee | ||
Comment 1•25 years ago
|
||
Urg I hadn't thought about that, sorry. The patch from the mail looks fine, how are you going to fix my kwindex bug? I just checked Rhino and it doesn't have the same problem because it does a complete separate lookup for the keyword.
Reporter | ||
Comment 2•25 years ago
|
||
The patch fixes the kwindex problem by treating any identifier that hadUnicodeEscape as non-reserved. /be
Reporter | ||
Comment 3•25 years ago
|
||
Reporter | ||
Comment 4•25 years ago
|
||
On second thought, I'm on sabbatical. Roger, can you get this into the trunk, and then we can figure out what branch to get it into for JS1.5? I don't see why the nscp_beta1_BRANCH needs it, but JS1.5 needs a branch. Maybe we should have a JS1_5_BRANCH. Whatever the resolution there, this change should go into the trunk ASAP. Can you get it in, r=brendan@mozilla.org? Thanks, /be
Assignee: brendan → rogerl
Status: ASSIGNED → NEW
Assignee | ||
Comment 5•25 years ago
|
||
Applied patch, ran tests. Checked in.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•