[FFmpeg-devel] [PATCH] lavc/rawdec: Only use AV_PIX_FMT_PAL8 for 1 bpp video

Mats Peterson matsp888 at yahoo.com
Sun Jan 31 16:05:10 CET 2016


On 01/31/2016 04:03 PM, Mats Peterson wrote:
> On 01/31/2016 03:59 PM, Michael Niedermayer wrote:
>> ffmpeg source is not unchanegable,
>> if you belive that pal8 should be used in place of monochrome you
>> could certaily make the code act more reasonable with a 2 entry
>> (black+white) PAL8
>>
>
> I don't believe. It's a fact. And it is used in other places (QuickTime,
> both raw and RLE, and BMP) for 1 bpp data, so why shouldn't it for AVI
> as well? Please, just apply my latest patch that restores the pal8-only
> behaviour. This is a lot more logical than switching to monow for the
> case of black & white, which has to detected with extraneous code, and
> which just complicates things in my book. The customer is always right,
> remember?
>

I already initialize the palette in raw_init_decode() to black & white 
for AVIs that don't have a palette.

Mats



More information about the ffmpeg-devel mailing list