1101037 - Fix dom/media/gmp compilation with mingw.

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Jacek Caban]

Attachment: fix

For scope_ptr.h the error is:

scoped_ptr.h:21:20: error: ‘operator=’ was not declared in this scope, and no declarations were found by argument-dependent lookup at the point of instantiation [-fpermissive]
      operator=(other);
                     ^
scoped_ptr.h:21:20: note: declarations in dependent base ‘scoped_array<wchar_t>’ are not found by unqualified lookup
scoped_ptr.h:21:20: note: use ‘this->operator=’ instead

Also GCC doesn't support unicode stream::open. UTF8 is also not perfect, but works... Also system includes should be lowercased for cross compilation.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 8524706 [details] [diff] [review]
fix

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

::: dom/media/gmp/GMPChild.cpp
@@ +315,3 @@
>    stream.open(path.get());
> +#else
> +  stream.open(NS_ConvertUTF16toUTF8(path).get());

Would just always doing this NS_ConvertUTF16toUTF8 Just Work? It's working for me here with MSVC. i.e. can we remove the above #ifdef _MSC_VER block?

r+ either way.

Comment 2

[Jacek Caban]

Technically, using UTF8 here is wrong. A better thing to do would be converting to system locale. That, however, brings another problem - Windows stores file names in UTF16, which may not be able to encode in current system locale. That's why using UTF16 here is better.

Since we don't have an option of using UTF16 on mingw in this case, I changed it to UTF8 just because we have a nice helper for that. I wanted to keep it simple to not complicate the code non-tier1 platform corner cases.

Comment 3

[Chris Pearce [:cpearce (Not reading bugmail)]]

OK. Best to leave it as is then. Thanks!

Comment 4

[Jacek Caban]

Thanks for review.

https://hg.mozilla.org/integration/mozilla-inbound/rev/62af62f4f0a4

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/62af62f4f0a4