diff mbox series

[FFmpeg-devel,2/2] avformat/mvdec: explicitly set duration

Message ID 20211218204628.22664-2-jpstewart@personalprojects.net
State Accepted
Commit 50bfd5e96e98e86df08cc7ffdf726f6ed6851ee7
Headers show
Series [FFmpeg-devel,1/2] avformat/mvdec: read frame rate from data stream | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
andriy/make_ppc success Make finished
andriy/make_fate_ppc success Make fate finished

Commit Message

John-Paul Stewart Dec. 18, 2021, 8:46 p.m. UTC
Resolves a warning that duration is being innaccurately estimated based
on bitrate.

Signed-off-by: John-Paul Stewart <jpstewart@personalprojects.net>
---
 libavformat/mvdec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Peter Ross Dec. 19, 2021, 3:16 a.m. UTC | #1
On Sat, Dec 18, 2021 at 03:46:28PM -0500, John-Paul Stewart wrote:
> Resolves a warning that duration is being innaccurately estimated based
> on bitrate.
> 
> Signed-off-by: John-Paul Stewart <jpstewart@personalprojects.net>
> ---
>  libavformat/mvdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/mvdec.c b/libavformat/mvdec.c
> index a5c5b205a6..1a5012e507 100644
> --- a/libavformat/mvdec.c
> +++ b/libavformat/mvdec.c
> @@ -319,7 +319,7 @@ static int mv_read_header(AVFormatContext *avctx)
>          avio_skip(pb, 4);
>          vst->codecpar->codec_type = AVMEDIA_TYPE_VIDEO;
>          vst->avg_frame_rate = fps;
> -        vst->nb_frames         = avio_rb32(pb);
> +        vst->duration = vst->nb_frames = avio_rb32(pb);
>          v = avio_rb32(pb);
>          switch (v) {
>          case 1:

both patches look good, and do not break tests.
i will apply in a couple of days.

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
diff mbox series

Patch

diff --git a/libavformat/mvdec.c b/libavformat/mvdec.c
index a5c5b205a6..1a5012e507 100644
--- a/libavformat/mvdec.c
+++ b/libavformat/mvdec.c
@@ -319,7 +319,7 @@  static int mv_read_header(AVFormatContext *avctx)
         avio_skip(pb, 4);
         vst->codecpar->codec_type = AVMEDIA_TYPE_VIDEO;
         vst->avg_frame_rate = fps;
-        vst->nb_frames         = avio_rb32(pb);
+        vst->duration = vst->nb_frames = avio_rb32(pb);
         v = avio_rb32(pb);
         switch (v) {
         case 1: