PeerConnectionImpl.cpp:369:1: warning: ‘void sipcc::Warn(JSContext*, const nsCString&)’ defined but not used [-Wunused-function]

RESOLVED FIXED in mozilla23

Status

()

defect
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

(Blocks 1 bug)

Trunk
mozilla23
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [WebRTC] [blocking-webrtc-] [qa-])

Attachments

(1 attachment)

GCC 4.8 build warning in mozilla-central:
{
media/webrtc/signaling/src/peerconnection/PeerConnectionImpl.cpp:369:1: warning: ‘void sipcc::Warn(JSContext*, const nsCString&)’ defined but not used [-Wunused-function]
}

The last usages of this function were removed in bug 855769's cset:
 https://hg.mozilla.org/mozilla-central/rev/915ad1e8a9bb#l3.22
Attachment #740946 - Flags: review?(jib)
Comment on attachment 740946 [details] [diff] [review]
fix: remove function

Review of attachment 740946 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm. Thanks!
[I assume that indicates an r+?  The flag is still set at "?", currently.]

Thanks!
Attachment #740946 - Flags: review?(jib) → review+
Yes sorry. :-)
Whiteboard: [WebRTC] [blocking-webrtc-]
https://hg.mozilla.org/mozilla-central/rev/5c7e9f21c920
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
Whiteboard: [WebRTC] [blocking-webrtc-] → [WebRTC] [blocking-webrtc-] [qa-]
You need to log in before you can comment on or make changes to this bug.