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

RESOLVED WONTFIX

Status

()

Core
XPCOM
RESOLVED WONTFIX
6 years ago
5 years ago

People

(Reporter: njn, Assigned: njn)

Tracking

Trunk
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

6 years ago
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.
(Assignee)

Comment 2

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