[FFmpeg-devel] [PATCH] add ivf muxer

[James Zern]

On Tue, Dec 14, 2010 at 11:23, Reimar D?ffinger
<Reimar.Doeffinger at gmx.de> wrote:
> On Tue, Dec 14, 2010 at 10:24:06AM +0100, Tomas H?rdin wrote:
>> On Mon, 2010-12-13 at 20:44 +0100, Reimar D?ffinger wrote:
> [...]
>
>> > + ? ?put_le64(pb, s->streams[0]->duration); // TODO: duration or
>> > number of frames?!?
>>
>> This sort of depends on whether VP8/IVF supports VFR, right? The demuxer
>> doesn't give many hints, but with CFR you'll have duration == number of
>> frames. Hm.
>
> Which means it does not at all depend on whether it supports VFR.
> If it doesn't it is as you said completely irrelevant, so going with
> this (simpler) solution is perfectly fine.
> My assumption is though that it does.
>
I think this was meant to be a frame count.