Hi, On Sun, Feb 5, 2012 at 11:53 PM, Anton Khirnov <[email protected]> wrote: > --- > libavcodec/imgconvert.c | 7 ++++++- > 1 files changed, 6 insertions(+), 1 deletions(-) > > diff --git a/libavcodec/imgconvert.c b/libavcodec/imgconvert.c > index d90254c..1fd211c 100644 > --- a/libavcodec/imgconvert.c > +++ b/libavcodec/imgconvert.c > @@ -460,7 +460,12 @@ int avpicture_layout(const AVPicture* src, enum > PixelFormat pix_fmt, int width, > } > } > > - if (desc->flags & PIX_FMT_PAL) > + if (desc->flags & PIX_FMT_PAL && > + !(pix_fmt == PIX_FMT_RGB8 || > + pix_fmt == PIX_FMT_BGR8 || > + pix_fmt == PIX_FMT_RGB4_BYTE || > + pix_fmt == PIX_FMT_BGR4_BYTE || > + pix_fmt == PIX_FMT_GRAY8)) > memcpy((unsigned char *)(((size_t)dest + 3) & ~3), src->data[1], 256 > * 4);
I don't think that's right. In practice, yes, they are paletted'ly implemented in Libav, just some muxers choose to not write the palette. What we need is a real patch that makes these formats unpaletted in Libav. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
