812278 - $OBJDIR/media/mtransport/standalone/*.cpp:87:131: warning: extra ‘;’ [-pedantic]

Status: RESOLVED FIXED

(Core :: WebRTC, defect)

Description

[Daniel Holbert [:dholbert]]

I get build warnings like:
> obj/media/mtransport/standalone/nricectx.cpp:87:131: warning: extra ‘;’ [-pedantic]
for most of the cpp files in media/mtransport/, ever since bug 809950 changed the definition of MOZ_MTLOG_MODULE to make it no longer require a semicolon after it.

We should drop all those no-longer-necessary semicolons to fix these warnings.

Comment 1

[Daniel Holbert [:dholbert]]

Attachment: fix

Comment 2

[Daniel Holbert [:dholbert]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/cc66f9bb1589

Comment 3

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/cc66f9bb1589

Comment 4

[Daniel Holbert [:dholbert]]

We don't build with -pedantic anymore, so we don't warn about this now, but just for consistency, I fixed 3 other instances of this that I ran across, for consistency:
 https://hg.mozilla.org/integration/mozilla-inbound/rev/30a252389a78

(Those were the last few MOZ_MTLOG_MODULE invocations w/ a trailing semicolon.)

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/30a252389a78