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

[Carl Eugen Hoyos]

Hi!

On 2007-08-30 10:32, Reimar D?ffinger wrote:
> 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.

Thank you, fixed.

> >  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?

There are no NAL_SVC_IDR in EXQI, so I wasn't sure if it would be
preferred to recognize such interesting samples.

New patch attached, Carl Eugen

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: patchSVC
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070830/5c4a274c/attachment.asc>