Closed Bug 791022 Opened 13 years ago Closed 12 years ago

GC: exact rooting API's need a shared way to avoid constantly rerooting constants

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: terrence, Unassigned)

References

Details

(Whiteboard: [js:t])

Constant Values such as undefined, null, true, false, etc do not need to be rooted. We should put pairs of |const Value fooValue = FooValue();| and |static HandleValue FooHandle() { return Handle::fromMarkedLocation(&fooValue); }| somewhere for this purpose. It should allow us to kill off many totally extraneous Rooted and will make it easier to convert remaining Values to Handles.
Whiteboard: [js:t]
Note that we now have JS::NullHandleValue and JS::UndefinedHandleValue in jsapi.h. We could do similar things for true and false.
Blocks: 773686
No longer blocks: ExactRooting
It looks like we did not have a serious need.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Resolution: FIXED → WONTFIX
You need to log in before you can comment on or make changes to this bug.