Closed
Bug 381301
Opened 17 years ago
Closed 17 years ago
Missing brace in uneval of object with a native-function getter
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: jruderman, Assigned: crowderbt)
References
Details
(Keywords: testcase)
Attachments
(1 file)
1.33 KB,
patch
|
mrbkap
:
review+
|
Details | Diff | Splinter Review |
js> uneval({x getter: print, y: 3}) ({get x () {[native code], y:3})
Reporter | ||
Comment 1•17 years ago
|
||
Oops, the "y: 3" isn't necessary to trigger the bug. js> uneval({x getter: print}) ({get x () {[native code]})
Assignee | ||
Comment 2•17 years ago
|
||
This prevents us from chomping an extra character at the end of the vchars string unless there was an opening '(' which we are chomping off the beginning to match the assumed ')'
Updated•17 years ago
|
Attachment #266506 -
Flags: review?(mrbkap) → review+
Assignee | ||
Comment 3•17 years ago
|
||
jsobj.c: 3.343
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 4•17 years ago
|
||
/cvsroot/mozilla/js/tests/js1_7/extensions/regress-381301.js,v <-- regress-381301.js initial revision: 1.1
Flags: in-testsuite+
Comment 5•17 years ago
|
||
/cvsroot/mozilla/js/tests/js1_7/extensions/regress-381301.js,v <-- regress-381301.js new revision: 1.2; previous revision: 1.1 use decodeURI since print is not native in the browser version of the test.
You need to log in
before you can comment on or make changes to this bug.
Description
•