[FFmpeg-devel] lavc/hevcdec: Silence warnings for DolbyVision

Submitted by Carl Eugen Hoyos on July 20, 2018, 12:18 a.m.

Details

Message ID CAB0OVGq=tmhZUO6PTSQ+iqWyNZFy6wWLfBjgj6nFa4Jx3Fi5xA@mail.gmail.com
State New
Headers show

Commit Message

Carl Eugen Hoyos July 20, 2018, 12:18 a.m.
Hi!

Attached patch silences several warnings shown for each frame in
DolbyVision streams, this time checking the whole separator as
suggested by Hendrik.

Please comment, Carl Eugen

Comments

James Almer July 20, 2018, 2:01 a.m.
On 7/19/2018 9:18 PM, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch silences several warnings shown for each frame in
> DolbyVision streams, this time checking the whole separator as
> suggested by Hendrik.
> 
> Please comment, Carl Eugen
> 
> 
> 0001-lavc-hevcdec-Silence-warnings-when-decoding-DolbyVis.patch
> 
> 
> From 9276b75de767dbccabb39d8dcaf5d373a1daa0ef Mon Sep 17 00:00:00 2001
> From: Carl Eugen Hoyos <ceffmpeg@gmail.com>
> Date: Fri, 20 Jul 2018 02:16:04 +0200
> Subject: [PATCH] lavc/hevcdec: Silence warnings when decoding DolbyVision.
> 
> ---
>  libavcodec/hevcdec.c |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c
> index 409e77f..4a047b8 100644
> --- a/libavcodec/hevcdec.c
> +++ b/libavcodec/hevcdec.c
> @@ -3000,6 +3000,10 @@ static int decode_nal_unit(HEVCContext *s, const H2645NAL *nal)
>      case HEVC_NAL_AUD:
>      case HEVC_NAL_FD_NUT:
>          break;
> +    case 62:
> +    case 63:
> +        if (!s->temporal_id) // DolbyVision
> +            break;
>      default:
>          av_log(s->avctx, AV_LOG_INFO,
>                 "Skipping NAL unit %d\n", s->nal_unit_type);

This message could also (or instead) be changed to verbose.

Patch hide | download patch | download mbox

From 9276b75de767dbccabb39d8dcaf5d373a1daa0ef Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos <ceffmpeg@gmail.com>
Date: Fri, 20 Jul 2018 02:16:04 +0200
Subject: [PATCH] lavc/hevcdec: Silence warnings when decoding DolbyVision.

---
 libavcodec/hevcdec.c |    4 ++++
 1 file changed, 4 insertions(+)

diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c
index 409e77f..4a047b8 100644
--- a/libavcodec/hevcdec.c
+++ b/libavcodec/hevcdec.c
@@ -3000,6 +3000,10 @@  static int decode_nal_unit(HEVCContext *s, const H2645NAL *nal)
     case HEVC_NAL_AUD:
     case HEVC_NAL_FD_NUT:
         break;
+    case 62:
+    case 63:
+        if (!s->temporal_id) // DolbyVision
+            break;
     default:
         av_log(s->avctx, AV_LOG_INFO,
                "Skipping NAL unit %d\n", s->nal_unit_type);
-- 
1.7.10.4