Re: [FFmpeg-devel] [PATCH 1/3] lavf/dv: do not set video timebase more than once

2023-04-27 Thread Anton Khirnov
Quoting Michael Niedermayer (2023-04-26 13:54:10) > On Mon, Apr 24, 2023 at 05:55:51PM +0200, Anton Khirnov wrote: > > Current code will call avpriv_set_pts_info() for each video frame, > > possibly setting a different timebase if the stream framerate changes. > > This violates API conventions, as

Re: [FFmpeg-devel] [PATCH 1/3] lavf/dv: do not set video timebase more than once

2023-04-26 Thread Michael Niedermayer
On Mon, Apr 24, 2023 at 05:55:51PM +0200, Anton Khirnov wrote: > Current code will call avpriv_set_pts_info() for each video frame, > possibly setting a different timebase if the stream framerate changes. > This violates API conventions, as the timebase is supposed to stay > constant after stream

[FFmpeg-devel] [PATCH 1/3] lavf/dv: do not set video timebase more than once

2023-04-24 Thread Anton Khirnov
Current code will call avpriv_set_pts_info() for each video frame, possibly setting a different timebase if the stream framerate changes. This violates API conventions, as the timebase is supposed to stay constant after stream creation. Change the demuxer to set a single timebase that is fine