[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:19:06 CET 2016


On 01/31/2016 04:05 PM, Mats Peterson wrote:
>> 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
>

People will have to get used to the video data being converted from 1 
bpp to 8 bpp internally, just as they have become used to 2 and 4 bpp 
data being converted to 8 bpp. It shouldn't come as a shock, since this 
is perfectly logical behaviour, as long as we don't have "pal1", "pal2" 
or "pal4" pixel formats.

Mats



More information about the ffmpeg-devel mailing list