821621 - Fix some warnings about unused WebRTC functions and variables

Status: RESOLVED FIXED

(Core :: WebRTC, defect)

Description

[Chris Peterson [:cpeterson]]

Attachment: fix-warnings.patch

When building with clang on OSX, I see a number of warnings about unused functions in .cpp files that #include voice_engine_defines.h:

media/webrtc/trunk/src/voice_engine/voice_engine_defines.h:493:10: warning: unused function 'Sleep' [-Wunused-function]
media/webrtc/trunk/src/voice_engine/voice_engine_defines.h:501:11: warning: unused function 'WebRtcTimeGetTime' [-Wunused-function]
media/webrtc/signaling/src/peerconnection/PeerConnectionCtx.cpp:36:8: warning: unused variable 'on' [-Wunused-variable]

The Sleep() and WebRtcTimeGetTime() functions are defined in an unnamed namespace. clang still complains if I make the functions inline. To suppress clang's warnings, I had to make the functions static inline.

Comment 1

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 692219 [details] [diff] [review]
fix-warnings.patch

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

I'm fine with the PeerConnectionCtx change

The voice_engines_defines: the bar is a bit higher there as it's imported code; we'd want to file an upstream bug (if it isn't already fixed).  r+, but please go to webrtc.org and file an 'issue' with a pointer to this bug/patch, and then add it as a comment here and make this bug dependent on the bug webrtc_updates

It's very easy to file a bug there.  If you really prefer not to, say so here and I'll do it.

Thanks for doing the cleanup!

Comment 2

[Chris Peterson [:cpeterson]]

(In reply to Randell Jesup [:jesup] from comment #1)
> The voice_engines_defines: the bar is a bit higher there as it's imported
> code; we'd want to file an upstream bug (if it isn't already fixed).  r+,
> but please go to webrtc.org and file an 'issue' with a pointer to this
> bug/patch, and then add it as a comment here and make this bug dependent on
> the bug webrtc_updates

No problem; I will file a bug upstream with the fix.

Comment 3

[Chris Peterson [:cpeterson]]

I landed the PeerConnectionCtx.cpp fix on m-i:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7c43da37e53f

I submitted the voice_engines_defines.h fixes upstream to webrtc:
https://code.google.com/p/webrtc/issues/detail?id=1228

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/7c43da37e53f