For all so-called NativeError.prototype objects, their "name" and "message" properties are enumerable when they shouldn't be. I am classifying this as minor since an initialization script can easily repair this using Object.defineProperty.
js> for (i in Error.prototype) print(i) name message fileName lineNumber
exn_enumerate() looks really weird and unnecessary. Sounds like we can easily fix this for FF5.
Missed when fixing bug 574867. /be
I'll try this.
Assignee: general → cam
Status: NEW → ASSIGNED
Looks like it landed: http://hg.mozilla.org/mozilla-central/rev/9743d95d473e
Target Milestone: --- → mozilla6
Version: unspecified → Trunk
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
The removed function ("exn_enumerate") is still declared at the top of jsexn.cpp (but never defined or used), which triggers: ../../../mozilla/js/src/jsexn.cpp:83: warning: ‘JSBool exn_enumerate(JSContext*, JSObject*)’ declared ‘static’ but never defined The attached trivial followup should fix this.
Attachment #526415 - Flags: review?(gal)
Comment on attachment 526415 [details] [diff] [review] followup to remove declaration of removed function Thanks.
Attachment #526415 - Flags: review?(gal) → review+
Landed followup: http://hg.mozilla.org/mozilla-central/rev/19f14915646a
You need to log in before you can comment on or make changes to this bug.