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)

x86
macOS
defect
Not set
minor

Tracking

()

VERIFIED FIXED

People

(Reporter: jruderman, Assigned: crowderbt)

References

Details

(Keywords: testcase)

Attachments

(1 file)

js> uneval({x getter: print, y: 3})
({get x () {[native code], y:3})
Oops, the "y: 3" isn't necessary to trigger the bug.

js> uneval({x getter: print})       
({get x () {[native code]})
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 ')'
Assignee: general → crowder
Status: NEW → ASSIGNED
Attachment #266506 - Flags: review?(mrbkap)
Attachment #266506 - Flags: review?(mrbkap) → review+
Blocks: 380581
jsobj.c: 3.343
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
/cvsroot/mozilla/js/tests/js1_7/extensions/regress-381301.js,v  <--  regress-381301.js
initial revision: 1.1
Flags: in-testsuite+
/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.
verified fixed 1.9.0 linux/mac*/windows.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: