Closed Bug 1174783 Opened 10 years ago Closed 10 years ago

Update the comment on DeferredFinalizeFunction to mention lifetime

Categories

(Core :: XPCOM, defect)

Product:
Core
Component:
XPCOM
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1174796
Tracking Flags:
Tracking Status
firefox41 --- affected

People

(Reporter: mccr8, Assigned: mccr8)

Details

In bug 861449 comment 22, dbaron pointed out that the comments for DeferredFinalize don't explicitly call out that you should delete any storage when you return true from the callback. Maybe a sentence like "If this function returns true, it will not be called again, so aData should be freed before returning."
I might as well just fix the comment at the same time as I fix bug 1174796.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.