Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-25 Thread James Almer
On 10/25/2020 10:26 AM, Anton Khirnov wrote: > Quoting Andreas Rheinhardt (2020-10-25 14:17:06) >> Anton Khirnov: >>> Quoting Michael Niedermayer (2020-10-21 13:17:46) are we talking about the same thing ? ./ffmpeg -i ~/tickets/4496/08_lect_01.rm file.avi produces a outpu

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-25 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2020-10-25 14:17:06) > Anton Khirnov: > > Quoting Michael Niedermayer (2020-10-21 13:17:46) > >> > >> are we talking about the same thing ? > >> ./ffmpeg -i ~/tickets/4496/08_lect_01.rm file.avi > >> > >> produces a output over 1 hour long before the patch, theres no s

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-25 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Michael Niedermayer (2020-10-21 13:17:46) >> >> are we talking about the same thing ? >> ./ffmpeg -i ~/tickets/4496/08_lect_01.rm file.avi >> >> produces a output over 1 hour long before the patch, theres no seek to >> before EOF >> >> ./ffmpeg -i ~/tickets/4496/08_lect

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-25 Thread Anton Khirnov
Quoting Michael Niedermayer (2020-10-21 13:17:46) > > are we talking about the same thing ? > ./ffmpeg -i ~/tickets/4496/08_lect_01.rm file.avi > > produces a output over 1 hour long before the patch, theres no seek to before > EOF > > ./ffmpeg -i ~/tickets/4496/08_lect_01.rm file-new.avi >

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-21 Thread James Almer
On 10/21/2020 8:17 AM, Michael Niedermayer wrote: > On Tue, Oct 20, 2020 at 01:41:50AM -0300, James Almer wrote: >> On 10/19/2020 11:42 AM, Anton Khirnov wrote: >>> Quoting James Almer (2020-10-14 19:26:51) On 10/14/2020 2:09 PM, Michael Niedermayer wrote: > On Wed, Oct 14, 2020 at 10:53:1

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-21 Thread Michael Niedermayer
On Tue, Oct 20, 2020 at 01:41:50AM -0300, James Almer wrote: > On 10/19/2020 11:42 AM, Anton Khirnov wrote: > > Quoting James Almer (2020-10-14 19:26:51) > >> On 10/14/2020 2:09 PM, Michael Niedermayer wrote: > >>> On Wed, Oct 14, 2020 at 10:53:10AM +0200, Anton Khirnov wrote: > Use a floating

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-19 Thread James Almer
On 10/19/2020 11:42 AM, Anton Khirnov wrote: > Quoting James Almer (2020-10-14 19:26:51) >> On 10/14/2020 2:09 PM, Michael Niedermayer wrote: >>> On Wed, Oct 14, 2020 at 10:53:10AM +0200, Anton Khirnov wrote: Use a floating-point score value to take into account bitrate, when multiple str

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-19 Thread Gyan Doshi
On 19-10-2020 08:14 pm, Anton Khirnov wrote: Quoting Gyan Doshi (2020-10-14 20:43:59) On 14-10-2020 02:23 pm, Anton Khirnov wrote: Use a floating-point score value to take into account bitrate, when multiple streams with the same resolution are present. Stop accessing private AVStream.codec

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-19 Thread Jean-Baptiste Kempf
On Mon, 19 Oct 2020, at 16:44, Anton Khirnov wrote: > > The saner way is to select best stream from each input and then select > > from that shortlist without considering disposition. > > Watches pelcome. I am not bringing about world peace here, I just want > ffmpeg.c to be a well-behaved user a

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-19 Thread Anton Khirnov
Quoting Gyan Doshi (2020-10-14 20:43:59) > > > On 14-10-2020 02:23 pm, Anton Khirnov wrote: > > Use a floating-point score value to take into account bitrate, when > > multiple streams with the same resolution are present. > > > > Stop accessing private AVStream.codec_info_nb_frames field, as the

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-19 Thread Anton Khirnov
Quoting James Almer (2020-10-14 19:26:51) > On 10/14/2020 2:09 PM, Michael Niedermayer wrote: > > On Wed, Oct 14, 2020 at 10:53:10AM +0200, Anton Khirnov wrote: > >> Use a floating-point score value to take into account bitrate, when > >> multiple streams with the same resolution are present. > >>

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-14 Thread Michael Niedermayer
On Wed, Oct 14, 2020 at 02:26:51PM -0300, James Almer wrote: > On 10/14/2020 2:09 PM, Michael Niedermayer wrote: > > On Wed, Oct 14, 2020 at 10:53:10AM +0200, Anton Khirnov wrote: > >> Use a floating-point score value to take into account bitrate, when > >> multiple streams with the same resolution

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-14 Thread Gyan Doshi
On 14-10-2020 02:23 pm, Anton Khirnov wrote: Use a floating-point score value to take into account bitrate, when multiple streams with the same resolution are present. Stop accessing private AVStream.codec_info_nb_frames field, as the sample in question http://samples.ffmpeg.org/ffmpeg-bugs/tr

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-14 Thread James Almer
On 10/14/2020 2:09 PM, Michael Niedermayer wrote: > On Wed, Oct 14, 2020 at 10:53:10AM +0200, Anton Khirnov wrote: >> Use a floating-point score value to take into account bitrate, when >> multiple streams with the same resolution are present. >> >> Stop accessing private AVStream.codec_info_nb_fra

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-14 Thread Michael Niedermayer
On Wed, Oct 14, 2020 at 10:53:10AM +0200, Anton Khirnov wrote: > Use a floating-point score value to take into account bitrate, when > multiple streams with the same resolution are present. > > Stop accessing private AVStream.codec_info_nb_frames field, as the > sample in question > http://samples

[FFmpeg-devel] [PATCH 2/3] ffmpeg.c: refine picking default video stream

2020-10-14 Thread Anton Khirnov
Use a floating-point score value to take into account bitrate, when multiple streams with the same resolution are present. Stop accessing private AVStream.codec_info_nb_frames field, as the sample in question http://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket2639/Thailand-Wave.wmv is now handled by