The default bug view has changed. See this FAQ.

Build failure in content/media/gstreamer/nsGStreamerReader.cpp when using clang

RESOLVED FIXED in mozilla16

Status

()

Core
Audio/Video
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: padenot, Assigned: padenot)

Tracking

unspecified
mozilla16
x86_64
Linux
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
Compilation fails when enabling gstreamer and using clang (version 3.0-6ubuntu3), with the following error :

http://pastebin.mozilla.org/1655529
(Assignee)

Comment 1

5 years ago
Created attachment 629982 [details] [diff] [review]
Possible patch.

Possible patch. Untested because of bug 761409.
(Assignee)

Comment 2

5 years ago
Comment on attachment 629982 [details] [diff] [review]
Possible patch.

I could test that applying the patch from bug 761030, seems to do fine.
Attachment #629982 - Flags: review?(kinetik)
Comment on attachment 629982 [details] [diff] [review]
Possible patch.

Thanks!
Attachment #629982 - Flags: review?(kinetik) → review+
Pasting the error here for anyone searching for this problem:

content/media/gstreamer/nsGStreamerReader.cpp:194:5: error: constant expression evaluates to -3 which cannot be narrowed to type 'guint' (aka 'unsigned int')
    ~GST_PLAY_FLAG_AUDIO, ~GST_PLAY_FLAG_VIDEO};
(Assignee)

Comment 5

5 years ago
Created attachment 635463 [details] [diff] [review]
(Added r=kinetik)
Assignee: nobody → paul
Attachment #629982 - Attachment is obsolete: true
Status: NEW → ASSIGNED
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/0da00a304a4a
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/0da00a304a4a
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.