diff mbox series

[FFmpeg-devel,1/5] avcodec/mlpdec: Set AV_FRAME_FLAG_KEY explicitly

Message ID GV1P250MB0737230EFB1A8FE7E765AE5E8FE62@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM
State Accepted
Commit 876a25027c59db38e3b7b0e70e3493fa49e76220
Headers show
Series [FFmpeg-devel,1/5] avcodec/mlpdec: Set AV_FRAME_FLAG_KEY explicitly | expand

Commit Message

Andreas Rheinhardt May 9, 2024, 1:52 a.m. UTC
It is currently always set for all audio frames, but this is
wrong (namely for MLP/TrueHD) and will be changed.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavcodec/mlpdec.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Andreas Rheinhardt May 17, 2024, 3:34 p.m. UTC | #1
Andreas Rheinhardt:
> It is currently always set for all audio frames, but this is
> wrong (namely for MLP/TrueHD) and will be changed.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>  libavcodec/mlpdec.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libavcodec/mlpdec.c b/libavcodec/mlpdec.c
> index 305c5d2b36..e85dac36a7 100644
> --- a/libavcodec/mlpdec.c
> +++ b/libavcodec/mlpdec.c
> @@ -1212,6 +1212,7 @@ static int read_access_unit(AVCodecContext *avctx, AVFrame *frame,
>              goto error;
>          m->is_major_sync_unit = 1;
>          header_size += m->major_sync_header_size;
> +        frame->flags |= AV_FRAME_FLAG_KEY;
>      }
>  
>      if (!m->params_valid) {

Will apply this patchset tomorrow unless there are objections.

- Andreas
diff mbox series

Patch

diff --git a/libavcodec/mlpdec.c b/libavcodec/mlpdec.c
index 305c5d2b36..e85dac36a7 100644
--- a/libavcodec/mlpdec.c
+++ b/libavcodec/mlpdec.c
@@ -1212,6 +1212,7 @@  static int read_access_unit(AVCodecContext *avctx, AVFrame *frame,
             goto error;
         m->is_major_sync_unit = 1;
         header_size += m->major_sync_header_size;
+        frame->flags |= AV_FRAME_FLAG_KEY;
     }
 
     if (!m->params_valid) {