[Bug 748877] theoraenc regression

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=748877

Tim-Philipp Müller <t.i.m at zen.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |t.i.m at zen.co.uk

--- Comment #2 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
This is caused by your patch:

commit c466ff474859cce91e9dd9b906c48f5fe0c4b164
Author: Nicola Murino <nicola.murino at gmail.com>
Date:   Tue Nov 18 16:52:40 2014 +0100

    matroskademux: set framerate 0/1 when duration is not known

    https://bugzilla.gnome.org/show_bug.cgi?id=740130



Ogg wants a fixed framerate for video input, so you need to put a videorate
before it. I don't think theoraenc's sink pad template is entirely right, I
think it should accept 0/1 framerates (for cases like RTP), and oggmux should
then advertise non-0/1 framerates.

In any case, this has likely led to broken video even before your patch, so it
needs to be fixed in your app/pipeline IMHO.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.