Closed Bug 826591 Opened 11 years ago Closed 11 years ago

Don't return an nsresult from NS_{R,Unr}egisterMemory{,Multi}Reporter

Categories

(Core :: XPCOM, defect)

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

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: n.nethercote, Assigned: n.nethercote)

Details

NS_{R,Unr}egisterMemory{,Multi}Reporter are called many times, but their return values are only checked twice, both in dom/workers/WorkerPrivate.cpp, and that check doesn't look that important.

Perhaps we should bother with a return value.
> Perhaps we should[n't] bother with a return value.

sgtm.
This doesn't seem worthwhile.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.