[FFmpeg-cvslog] libavcodec/zmbv: change 24-bit decoder channel order, from RGB24 to BGR24

Matthew Fearnley git at videolan.org
Tue Apr 2 18:47:43 EEST 2019


ffmpeg | branch: master | Matthew Fearnley <matthew.w.fearnley at gmail.com> | Fri Mar 29 22:03:03 2019 +0000| [1046e880884bb5f0da4fb7d50028ff599550245c] | committer: Tomas Härdin

libavcodec/zmbv: change 24-bit decoder channel order, from RGB24 to BGR24

This brings the channel order in line with that used in 32-bit mode (BGR0).

24-bit decoding is disabled by default (#ifdef ZMBV_ENABLE_24BPP), and no
prior encoders or sample videos are known to exist for this bit depth, so
I consider this change in implementation is unlikely to affect anyone.

The decision has been made in agreement with the DOSBox Development Team
(dosbox.crew at gmail.com), specifically with harekiet, who wrote the original
codec.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=1046e880884bb5f0da4fb7d50028ff599550245c
---

 libavcodec/zmbv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/zmbv.c b/libavcodec/zmbv.c
index 71ec2cd424..898b62d065 100644
--- a/libavcodec/zmbv.c
+++ b/libavcodec/zmbv.c
@@ -473,7 +473,7 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, AVPac
             c->bpp = 24;
             decode_intra = zmbv_decode_intra;
             c->decode_xor = zmbv_decode_xor_24;
-            avctx->pix_fmt = AV_PIX_FMT_RGB24;
+            avctx->pix_fmt = AV_PIX_FMT_BGR24;
             c->stride = c->width * 3;
             break;
 #endif //ZMBV_ENABLE_24BPP



More information about the ffmpeg-cvslog mailing list