Re: [FFmpeg-devel] [PATCH] avcodec: add Mandsoft Screen Capture Codec decoder

2017-04-15 Thread Paul B Mahol
On 4/15/17, Carl Eugen Hoyos wrote: > 2017-04-15 13:49 GMT+02:00 Paul B Mahol : > >> +case 32: avctx->pix_fmt = AV_PIX_FMT_BGRA; break; > > (I am thankful for this patch and, yes, I believe we had this > discussion before.) > If this is a screen capture codec, I believe BGR0 is the correct >

Re: [FFmpeg-devel] [PATCH] avcodec: add Mandsoft Screen Capture Codec decoder

2017-04-15 Thread Carl Eugen Hoyos
2017-04-15 13:49 GMT+02:00 Paul B Mahol : > +case 32: avctx->pix_fmt = AV_PIX_FMT_BGRA; break; (I am thankful for this patch and, yes, I believe we had this discussion before.) If this is a screen capture codec, I believe BGR0 is the correct colorspace. I understand that you tested that the

[FFmpeg-devel] [PATCH] avcodec: add Mandsoft Screen Capture Codec decoder

2017-04-15 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- configure | 1 + doc/general.texi| 1 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 ++ libavcodec/mscc.c | 220