[FFmpeg-devel] [PATCH] Silence warnings about NAL_SVC_NON_IDR in H264

[Reimar Döffinger]

Hello,
On Thu, Aug 30, 2007 at 12:18:25PM +0200, Carl Eugen Hoyos wrote:
> Hi!
> 
> When watching EXQI, a HDTV program broadcasting in H264, with MPlayer,
> this warning appears every few seconds:
> 
> [h264 @ 0x891c570]Unknown NAL code: 20 (18 bits)
> 
> Attached patch silences the warnings by ignoring these NALs.
> 
> Carl Eugen

> Index: libavcodec/h264data.h
> ===================================================================
> --- libavcodec/h264data.h	(Revision 10260)
> +++ libavcodec/h264data.h	(Arbeitskopie)
> @@ -74,7 +74,9 @@
>  NAL_END_STREAM,
>  NAL_FILLER_DATA,
>  NAL_SPS_EXT,
> -NAL_AUXILIARY_SLICE=19
> +NAL_AUXILIARY_SLICE=19,
> +NAL_SVC_NON_IDR,
> +NAL_SVC_IDR

I'd be for adding a ',' here as well, so the list can be extended
without having to change that line.

>  static const AVRational pixel_aspect[14]={
> Index: libavcodec/h264.c
> ===================================================================
> --- libavcodec/h264.c	(Revision 10260)
> +++ libavcodec/h264.c	(Arbeitskopie)
> @@ -7846,6 +7846,7 @@
>          case NAL_FILLER_DATA:
>          case NAL_SPS_EXT:
>          case NAL_AUXILIARY_SLICE:
> +        case NAL_SVC_NON_IDR:
>              break;

Hmm... Did you want to add NAL_SVC_IDR here as well?

Greetings,
Reimar D?ffinger