Closed Bug 660922 Opened 9 years ago Closed 8 years ago

Unused result of SendPMemoryReportRequestConstructor

Categories

(Core :: XPCOM, defect)

x86
Linux
defect
Not set

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: jdm, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [build_warning] [good first bug] [mentor=jdm] [lang=c++])

/home/t_mattjo/src/firefox/mobilebase/dom/ipc/ContentParent.cpp: In member function ‘virtual nsresult mozilla::dom::ContentParent::Observe(nsISupports*, const char*, const PRUnichar*)’:
/home/t_mattjo/src/firefox/mobilebase/dom/ipc/ContentParent.cpp:624: warning: ignoring return value of ‘mozilla::dom::PContentParent::PMemoryReportRequestParent* mozilla::dom::PContentParent::SendPMemoryReportRequestConstructor()’, declared with attribute warn_unused_result

We should probably just pop a |mozilla::unused <<| in front of the call.
This might be an interesting error, not sure.
Component: IPC → XPCOM
QA Contact: ipc → xpcom
Chris: what if we called SetChildMemoryReporters(emptyTArray) in the case of failure here? Does that seem sensible?
Whiteboard: [build_warning] [good first bug] [mentor=jdm] → [build_warning] [good first bug] [mentor=jdm] [lang=c++]
Warning is gone.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.