[vlc-devel] [PATCH] opus encoder: do not write a full blown ogg header in extra data

[Tristan Matthews]

On Fri, Jan 17, 2014 at 10:56 AM, Denis Charmet <typx at dinauz.org> wrote:

> Hi,
>
> I'd say that it looks good to me but are you sure that it doesn't break
> ogg muxing?
>

This does break ogg muxing:
[00007f31e0003418] mux_ogg mux debug: creating header for Opus
[00007f31e0003418] mux_ogg mux error: header data corrupted
[00007f31e0003418] mux_ogg mux debug: data starts from 28

Muxing with avformat yields a playable file (with this patch), but what
should be a 2 minute file shows up incorrectly as a 19 hour file.
opusinfo (from opus-tools) is a little more informative:
WARNING: Sample count ahead of granule (145920>145919) in stream 1
(repeated)

Best,
Tristan

-- 
Tristan Matthews
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20140117/5c16b7fb/attachment.html>