[FFmpeg-devel] [PATCH] avcodec/h264_parse: no need check ref list1 for P slices.

[Lin, Decai]

> -----Original Message-----
> From: ffmpeg-devel [mailto:ffmpeg-devel-bounces at ffmpeg.org] On Behalf Of
> mypopy at gmail.com
> Sent: 2019年2月1日 15:14
> To: FFmpeg development discussions and patches
> <ffmpeg-devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH] avcodec/h264_parse: no need check ref
> list1 for P slices.
> 
> On Fri, Feb 1, 2019 at 1:28 PM Lin, Decai <decai.lin at intel.com> wrote:
> >
> >
> >
> > > -----Original Message-----
> > > From: ffmpeg-devel [mailto:ffmpeg-devel-bounces at ffmpeg.org] On
> > > Behalf Of Michael Niedermayer
> > > Sent: 2019年2月1日 1:12
> > > To: FFmpeg development discussions and patches
> > > <ffmpeg-devel at ffmpeg.org>
> > > Subject: Re: [FFmpeg-devel] [PATCH] avcodec/h264_parse: no need
> > > check ref
> > > list1 for P slices.
> > >
> > > On Thu, Jan 31, 2019 at 03:36:56PM +0800, Decai Lin wrote:
> > > > This is robust for some corner case there is incorrect list1 count
> > > > in pps header, but it's a P slice and can be decoded well.
> > >
> > > please provide a sample h264 video that needs this
> > >
> >
> > Attached the test clip for this patch.
> >
> 
> I think the better way is to add a new fate test case with the test clip for
> h264dec, isn't it?

Yes, I can file another patch for the related fate test case.

Thanks