[FFmpeg-devel] [PATCH 4/4] dpx: fix endianess for RGB 8bits

[Michael Niedermayer]

On Wed, Aug 13, 2014 at 10:21:54AM +0000, Christophe Gisquet wrote:
> Fixes DLAD_8b_3c_big.dpx from ticket #3692
> ---
>  libavcodec/dpx.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c
> index 2ad7527..d4d6833 100644
> --- a/libavcodec/dpx.c
> +++ b/libavcodec/dpx.c
> @@ -232,6 +232,8 @@ static int decode_frame(AVCodecContext *avctx,
>          avctx->pix_fmt = AV_PIX_FMT_GRAY8;
>          break;
>      case 50081:
> +        avctx->pix_fmt = AV_PIX_FMT_BGR24;
> +        break;

this possibly breaks decoding of
checkerboard_1080p_nuke_bigendian_8bit_noalpha.dpx
the cross in the middle is displayed as cyan while the other samples
have it yellow

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140814/ee2a16c5/attachment.asc>