Closed Bug 752093 Opened 12 years ago Closed 12 years ago

Add rooters to jsreflect.cpp

Categories

(Core :: JavaScript Engine, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 782802

People

(Reporter: billm, Unassigned)

References

()

Details

(Whiteboard: [js:t])

In general, local variables of type T need to be converted to RootedVar<T>s. Parameters need to be converted to Handle<T>.

We have a dynamic analysis to check if the rooting is done correctly. Talk to Terrence or Brian to figure out how to get it running.
Added the internal rooting docs to the URL.

Also for reference, I implemented an earlier version of Reflect.parse with explicit rooting. See bug 533874 comment 39, or

    https://bug533874.bugzilla.mozilla.org/attachment.cgi?id=452356

Dave
Whiteboard: [js:t]
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.