warning: extra ';' [-pedantic] in media/mtransport/

RESOLVED FIXED in mozilla19

Status

()

Core
WebRTC
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

(Blocks: 1 bug)

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

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Newish build warnings:

Warning: -pedantic in /mozilla/media/mtransport/nricectx.h: extra ';'
/mozilla/media/mtransport/nricectx.h:129:880: warning: extra ';' [-pedantic]
Warning: -pedantic in /mozilla/media/mtransport/nricemediastream.h: extra ';'
/mozilla/media/mtransport/nricemediastream.h:97:910: warning: extra ';' [-pedantic]

Looks like this is from including ";" after a macro that already includes a ";" inside of it (or otherwise doesn't need a ";")
(Assignee)

Updated

6 years ago
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
(Assignee)

Comment 1

6 years ago
Created attachment 670998 [details] [diff] [review]
fix: Remove unnecessary semicolon after NS_INLINE_DECL_THREADSAFE_REFCOUNTING
Attachment #670998 - Flags: review?(rjesup)

Updated

6 years ago
Attachment #670998 - Flags: review?(rjesup) → review+
(Assignee)

Comment 2

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/637ab1786ee1
OS: Linux → All
Hardware: x86_64 → All
https://hg.mozilla.org/mozilla-central/rev/637ab1786ee1
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla19

Updated

6 years ago
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.