[vlc-devel] [PATCH] opus: add opus encoder

[Timothy B. Terriberry]

I haven't reviewed the whole patch, but a couple of comments:

Denis Charmet wrote:
> You are assuming little endian running host what about a big endian?
> (yes it is rare but still).

These are for reading/writing values from the headers, no? Those fields 
are defined to be little-endian (c.f. 
<http://tools.ietf.org/html/draft-ietf-codec-oggopus#section-5>).

>> +    const char *opus_version = opus_get_version_string();
> Is that an allocated string? if yes don't you need a free?

No, it's a static constant.