On 02/19/2012 09:44 AM, Anton Khirnov wrote:
> ---
> libavcodec/targaenc.c | 56 ++++++++++++++++++++++++++----------------------
> 1 files changed, 30 insertions(+), 26 deletions(-)
>
[...]
> switch(avctx->pix_fmt) {
> case PIX_FMT_GRAY8:
> - outbuf[2] = TGA_BW; /* uncompressed grayscale image */
> - outbuf[16] = 8; /* bpp */
> + pkt->data[2] = TGA_BW; /* uncompressed grayscale image */
> + pkt->data[16] = 8; /* bpp */
> break;
> case PIX_FMT_RGB555LE:
> - outbuf[2] = TGA_RGB; /* uncompresses true-color image */
> - outbuf[16] = 16; /* bpp */
> + pkt->data[2] = TGA_RGB; /* uncompresses true-color image */
> + pkt->data[16] = 16; /* bpp */
> break;
> case PIX_FMT_BGR24:
> - outbuf[2] = TGA_RGB; /* uncompressed true-color image */
> - outbuf[16] = 24; /* bpp */
> + pkt->data[2] = TGA_RGB; /* uncompressed true-color image */
> + pkt->data[16] = 24; /* bpp */
> break;
> case PIX_FMT_BGRA:
> - outbuf[2] = TGA_RGB; /* uncompressed true-color image */
> - outbuf[16] = 32; /* bpp */
> + pkt->data[2] = TGA_RGB; /* uncompressed true-color image */
> + pkt->data[16] = 32; /* bpp */
could probably use some vertical alignment here.
the rest looks good.
-Justin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
