[Ffmpeg-devel] [PATCH] add support for indexed 8bpp BMP files

lode leroy lode_leroy
Mon Apr 16 13:33:47 CEST 2007


Hi
On Fri Apr 6 19:03:11 CEST 2007, Michael Niedermayer wrote:
>On Fri, Apr 06, 2007 at 04:54:28PM +0200, lode leroy wrote:
> > This patch adds 8bpp support to libavcodec's bmp.c
>tabs and trailing whitespace are forbidden in svn

I fixed the whitespace issues

> > +	if (avctx->pix_fmt == PIX_FMT_PAL8) {
> > +        av_log(avctx, AV_LOG_ERROR, "copying palette with %d colors\n", 
>clrimportant);
>whats this?

euh... nothing.... (removed it)

> > +	    > +	    memcpy(p->data[1], pal, clrimportant*sizeof(uint32_t));
>likely wrong on big endian

I think it isn't. it's an array of R / G / B / 0 bytes
but I did change the sizeof(uint32_t) to "4" ...

-- lode

_________________________________________________________________
All things trendy for Windows Live Messenger ... 
http://entertainment.msn.be/funwithmessenger
-------------- next part --------------
A non-text attachment was scrubbed...
Name: libavcodec-bmp-8bpp.diff
Type: application/octet-stream
Size: 1513 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070416/2dcff993/attachment.obj>



More information about the ffmpeg-devel mailing list