On 22/07/15 14:23, Vittorio Giovara wrote: > On Tue, Jul 21, 2015 at 7:43 AM, Luca Barbato <[email protected]> wrote: >> On 21/07/15 00:47, Vittorio Giovara wrote: >>> From: Michael Niedermayer <[email protected]> >>> >>> Found-By: ami_stuff >>> Signed-off-by: Vittorio Giovara <[email protected]> >>> --- >>> libavcodec/dds.c | 12 +++++++----- >>> 1 file changed, 7 insertions(+), 5 deletions(-) >>> >>> diff --git a/libavcodec/dds.c b/libavcodec/dds.c >>> index 19f7dd0..1266ef2 100644 >>> --- a/libavcodec/dds.c >>> +++ b/libavcodec/dds.c >>> @@ -357,13 +357,13 @@ static int parse_pixel_format(AVCodecContext *avctx) >>> avctx->pix_fmt = AV_PIX_FMT_BGR24; >>> /* 32 bpp */ >>> else if (bpp == 32 && r == 0xff0000 && g == 0xff00 && b == 0xff && >>> a == 0) >>> - avctx->pix_fmt = AV_PIX_FMT_RGBA; // opaque >>> - else if (bpp == 32 && r == 0xff && g == 0xff00 && b == 0xff0000 && >>> a == 0) >>> avctx->pix_fmt = AV_PIX_FMT_BGRA; // opaque >>> + else if (bpp == 32 && r == 0xff && g == 0xff00 && b == 0xff0000 && >>> a == 0) >>> + avctx->pix_fmt = AV_PIX_FMT_RGBA; // opaque >>> else if (bpp == 32 && r == 0xff0000 && g == 0xff00 && b == 0xff && >>> a == 0xff000000) >>> - avctx->pix_fmt = AV_PIX_FMT_RGBA; >>> - else if (bpp == 32 && r == 0xff && g == 0xff00 && b == 0xff0000 && >>> a == 0xff000000) >>> avctx->pix_fmt = AV_PIX_FMT_BGRA; >>> + else if (bpp == 32 && r == 0xff && g == 0xff00 && b == 0xff0000 && >>> a == 0xff000000) >>> + avctx->pix_fmt = AV_PIX_FMT_RGBA; >> >> RGBA -> BGRA basically? > > yes
Probably can be simplified a little, but can happen later. > Not to my knowledge, I could maybe change it to check for postproc == > NONE if you think it's better. Sure. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
