On Sun, 16 Jul 2017 14:27:50 +0200
Luca Barbato <lu_z...@gentoo.org> wrote:

> On 7/16/17 2:12 PM, wm4 wrote:
> > On Sun, 16 Jul 2017 13:46:19 +0200
> > Luca Barbato <lu_z...@gentoo.org> wrote:
> >   
> >> ---
> >>  libavcodec/pngdec.c | 2 ++
> >>  1 file changed, 2 insertions(+)
> >>
> >> diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
> >> index 927248f2e3..6be463344a 100644
> >> --- a/libavcodec/pngdec.c
> >> +++ b/libavcodec/pngdec.c
> >> @@ -521,6 +521,8 @@ static int decode_frame(AVCodecContext *avctx,
> >>                             s->color_type == PNG_COLOR_TYPE_GRAY_ALPHA) {
> >>                      avctx->pix_fmt = AV_PIX_FMT_YA16BE;
> >>                  } else {
> >> +                    avpriv_request_sample(avctx, "Bit depth %d color type 
> >> %d",
> >> +                                          s->bit_depth, s->color_type);
> >>                      goto fail;
> >>                  }
> >>    
> > 
> > I don't think this is needed. PNG sample files for all possible format
> > variations are freely available or can be easily produced.  
> 
> Locally changed to avpriv_report_missing_feature, is that ok for you?

Yeah... but the sample argument should probably be set to 0.
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to