[FFmpeg-devel] [PATCH v2] avformat/ivfenc: Set the "number of frames" in IVF header

[Dai, Jianhui J]

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of
> Anton Khirnov
> Sent: Sunday, July 2, 2023 8:59 PM
> To: FFmpeg development discussions and patches <ffmpeg-
> devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH v2] avformat/ivfenc: Set the "number of
> frames" in IVF header
> 
> Quoting Ronald S. Bultje (2023-07-02 14:44:42)
> > Hi,
> >
> > On Sun, Jul 2, 2023 at 5:35 AM Anton Khirnov <anton at khirnov.net> wrote:
> >
> > > Quoting Ronald S. Bultje (2023-07-02 02:03:11)
> > > > I see this discussion now...
> > > >
> > > > I don't think I agree with the above. First of all, IVF has two
> > > > fields there (it seems): duration, and n_frames.
> > >
> > > Got any source for that? I only see one field in
> > > https://wiki.multimedia.cx/index.php/Duck_IVF
> > >
> >
> > This somewhat-widely used tool called "ffmpeg" has used it like that
> > for the past, say, 10 years. I've heard some projects or companies
> > around the world use this tool, and backwards compatibility is supposedly
> a thing.
> 
> Yeah yeah very funny.
> 
> You claimed there are two fields. My question was where is this second field
> supposed to be.
> 

Thank for all your commentary.
Let me update the V4 to set both the `nb_frames` and `duration` in IVF demux. Wish it could be both spec compatible and backward compatible.

> --
> Anton Khirnov
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email ffmpeg-devel-request at ffmpeg.org
> with subject "unsubscribe".