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

RESOLVED FIXED in mozilla19

Status

()

Product:
Core
Component:
WebRTC
Status:
RESOLVED FIXED
Reported:
6 years ago
Modified:
6 years ago

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

(Blocks: 1 bug)

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

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

fix: Remove unnecessary semicolon after NS_INLINE_DECL_THREADSAFE_REFCOUNTING
1.89 KB, patch
jesup
: review+
Details | Diff | Splinter Review
(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: 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+

Comment 3

•
6 years ago 
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.