[FFmpeg-devel] [PATCH 1/2] avformat/mov: always parse video headers

[Nicolas Gaullier]

>De : Paul B Mahol <onemda at gmail.com> 
>Envoyé : vendredi 29 janvier 2021 23:46
>À : FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
>Cc : Nicolas Gaullier <nicolas.gaullier at cji.paris>
>Objet : Re: [FFmpeg-devel] [PATCH 1/2] avformat/mov: always parse video headers
>
>Why is AV1 touched?
Well, AV1 is supposed to be left untouched but follows now the general case (except HEVC which has rich extradata and don't need "deep" parsing).
But sorry, I realized with patchwork that I forgot to run fate with SAMPLES set... and so at the end, I have to fix fate results for two items
mov-init-nonkeyframe
mov-zombie
I will take some time next week to check the reason why before commiting the diff (or fixing my patch if there were a real issue).

Nicolas
>On Fri, Jan 29, 2021 at 6:24 PM Nicolas Gaullier <mailto:nicolas.gaullier at cji.paris> wrote:
>This is aligned with current code of matroskadec.
>Fixes field_order reported by ffprobe for MP4 H264, for example.
>---
> libavformat/mov.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
>diff --git a/libavformat/mov.c b/libavformat/mov.c
>index ad0b9a58d3..b5e012190b 100644
>--- a/libavformat/mov.c
>+++ b/libavformat/mov.c
>@@ -2461,12 +2461,13 @@ static int mov_finalize_stsd_codec(MOVContext *c, AVIOContext *pb,
>     case AV_CODEC_ID_VP9:
>         st->need_parsing = AVSTREAM_PARSE_FULL;
>         break;
>-    case AV_CODEC_ID_AV1:
>-        st->need_parsing = AVSTREAM_PARSE_HEADERS;
>-        break;
>     default:
>         break;
>     }
>+    if (!st->need_parsing && st->codecpar->codec_type == AVMEDIA_TYPE_VIDEO
>+        && st->codecpar->codec_id != AV_CODEC_ID_HEVC)
>+        st->need_parsing = AVSTREAM_PARSE_HEADERS;
>+
>     return 0;
> }
>
>-- 
>2.27.0.windows.1
>
>_______________________________________________
>ffmpeg-devel mailing list
>mailto:ffmpeg-devel at ffmpeg.org
>https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>To unsubscribe, visit link above, or email
>mailto:ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".