922664 - Build omx-plugin in C++11 mode

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[(no longer active)]

Attachment: Patch (v1)

Spinoff from bug 920292.

Comment 1

[(no longer active)]

https://tbpl.mozilla.org/?tree=Try&rev=1520a373c87e

Comment 2

[cajbir (:cajbir)]

Comment on attachment 812656 [details] [diff] [review]
Patch (v1)

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

Please do one of the following:

1) Update media/omx-plugin/incude/ics/README_MOZILLA, update.sh and update.patch to include your changes so when updated with Android source your changes are included; or
2) Update README_MOZILLA explaining where the original source came from and that patches have been applied. Delete update.sh and update.patch. We'll lose the ability to easily provide updates here.

I'd opt for (2) since it's easier and we'll probably never need to update. If we do, we can diff.

Comment 3

[(no longer active)]

Attachment: Patch (v2)

Here's option #2.  :-)

Comment 4

[(no longer active)]

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

Comment 5

[Phil Ringnalda (:philor)]

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