[libav-devel] [PATCH 1/3] flvdec: Export flv flags for nellymoser through side data
lu_zero at gentoo.org
Tue Dec 13 16:02:09 CET 2011
On 13/12/11 15:52, Martin Storsjö wrote:
> On Tue, 13 Dec 2011, Justin Ruggles wrote:
>> On 12/13/2011 09:06 AM, Martin Storsjö wrote:
>>> On Tue, 13 Dec 2011, Justin Ruggles wrote:
>>>> I can't think of anything better right now, but it does seem a little
>>>> odd to me. Before decoding, the sample_rate is set by the user (via the
>>>> flv demuxer). The nellymoser decoder doesn't currently touch
>>>> sample_rate. If the point of the flag is to have the change come from
>>>> the decoder, what advantage does that have over the demuxer changing
>>>> sample_rate directly? I know that the user changing certain fields
>>>> init() will lead to bad things sometimes, but we don't have any
>>>> documented rules about such things do we?
>>> The advantage is that sample_rate is updated only once you actually
>>> that packet - you might demux and fill a buffer of packets, but the
>>> rate change isn't made effective until you actually decode them,
>>> you'd get wrong sample rate for a few of the preceding packets.
>> indeed. sounds like an alright solution then.
> Ok, I'll push this in a little while then.
Just make so the flag is generic enough to be reused. new extradata
sounds generic enough ^^; (maybe a bit too much).
More information about the libav-devel