Fix some JSFunctionSpec definitions in nsJSEnvironment.cpp

RESOLVED FIXED

Status

()

Core
DOM
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Robert Sayre, Assigned: Robert Sayre)

Tracking

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

8 years ago
these broke with the JM merge.
(Assignee)

Comment 1

8 years ago
Created attachment 474403 [details] [diff] [review]
change to two instead of three
Attachment #474403 - Flags: review?
Comment on attachment 474403 [details] [diff] [review]
change to two instead of three

r=dbaron

(I just wrote pretty much the same patch.)
Attachment #474403 - Flags: review? → review+
(Assignee)

Updated

8 years ago
Assignee: nobody → sayrer
(Assignee)

Comment 3

8 years ago
http://hg.mozilla.org/mozilla-central/rev/ac779114cfa6
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Created attachment 474407 [details] [diff] [review]
more

More bustage, currently untested, but some parts of it will be soon (though others I can't test).
(Assignee)

Updated

8 years ago
Attachment #474407 - Flags: review+
We should use the JS_FS and JS_FN macros (JS_FS changed, though, so it wouldn't have helped -- but it might next time).

/be
Me three. See bug 583143, which I'm not marking as a dupe because I think there are some pieces still missing.
You need to log in before you can comment on or make changes to this bug.