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

2023-07-02 Thread Dai, Jianhui J
> -Original Message- > From: ffmpeg-devel On Behalf Of > Anton Khirnov > Sent: Sunday, July 2, 2023 8:59 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH v2] avformat/ivfenc: Set the "number

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

2023-07-02 Thread Ronald S. Bultje
Hi, On Sun, Jul 2, 2023 at 8:59 AM Anton Khirnov wrote: > Quoting Ronald S. Bultje (2023-07-02 14:44:42) > > Hi, > > > > On Sun, Jul 2, 2023 at 5:35 AM Anton Khirnov wrote: > > > > > Quoting Ronald S. Bultje (2023-07-02 02:03:11) > > > > I see this discussion now... > > > > > > > > I don't thin

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

2023-07-02 Thread Anton Khirnov
Quoting Ronald S. Bultje (2023-07-02 14:44:42) > Hi, > > On Sun, Jul 2, 2023 at 5:35 AM Anton Khirnov 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

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

2023-07-02 Thread Ronald S. Bultje
Hi, On Sun, Jul 2, 2023 at 5:35 AM Anton Khirnov 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 on

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

2023-07-02 Thread Anton Khirnov
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 --

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

2023-07-01 Thread Ronald S. Bultje
28, 2023 11:25 PM > > > To: ffmpeg-devel@ffmpeg.org > > > Subject: Re: [FFmpeg-devel] [PATCH v2] avformat/ivfenc: Set the > "number of > > > frames" in IVF header > > > > > > Quoting Dai, Jianhui J (2023-06-05 02:53:35) > > > > diff --git a/l

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

2023-06-29 Thread Anton Khirnov
Quoting Dai, Jianhui J (2023-06-29 08:03:18) > > > > -Original Message- > > From: ffmpeg-devel On Behalf Of > > Anton Khirnov > > Sent: Wednesday, June 28, 2023 11:25 PM > > To: ffmpeg-devel@ffmpeg.org > > Subject: Re: [FFmpeg-devel] [PAT

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

2023-06-28 Thread Dai, Jianhui J
> -Original Message- > From: ffmpeg-devel On Behalf Of > Anton Khirnov > Sent: Wednesday, June 28, 2023 11:25 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v2] avformat/ivfenc: Set the "number of > frames" in IVF header > >

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

2023-06-28 Thread Anton Khirnov
Quoting Dai, Jianhui J (2023-06-05 02:53:35) > diff --git a/libavformat/ivfdec.c b/libavformat/ivfdec.c > index 511f2387ed..01012db948 100644 > --- a/libavformat/ivfdec.c > +++ b/libavformat/ivfdec.c > @@ -53,6 +53,7 @@ static int read_header(AVFormatContext *s) > st->codecpar->height = av

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

2023-06-19 Thread Dai, Jianhui J
> -Original Message- > From: Dai, Jianhui J > Sent: Monday, June 5, 2023 8:54 AM > To: ffmpeg-devel@ffmpeg.org > Subject: [PATCH v2] avformat/ivfenc: Set the "number of frames" in IVF header > > Should set "number of frames" to bytes 24-27 of IVF header, not duration. > It is described

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

2023-06-04 Thread Dai, Jianhui J
Should set "number of frames" to bytes 24-27 of IVF header, not duration. It is described by [1], and confirmed by parsing all IVF files in [2]. This commit also updates the md5sum of refs to pass fate-cbs. [1] Duck IVF - MultimediaWiki https://wiki.multimedia.cx/index.php/Duck_IVF [2] webm/vp8-