[FFmpeg-devel] [PATCH] lavc/rawdec: Remove superfluous AV_PIX_FMT_MONOWHITE code

Mats Peterson matsp888 at yahoo.com
Sun Jan 31 12:19:21 CET 2016


On 01/31/2016 11:31 AM, Michael Niedermayer wrote:
> On Sun, Jan 31, 2016 at 01:34:46AM +0100, Mats Peterson wrote:
>> On 01/30/2016 05:10 AM, Mats Peterson wrote:
>>> Now when both AVI and QuickTime use pal8 for 1 bpp video, there's no
>>> need to keep the monow stuff.
>>>
>> I should add that I'm only removing stuff that I've added myself
>> before, so don't worry.
>
> should this still be applied after
> 6ffac5d33d334f847150356293ecb6f9eaf69e15 ?
>
> [...]
>
>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

I've obviously missed that one, Michael, but I don't see the reason to 
switch to monow whatsoever. The space saving of using monow rather than 
pal8 for 1 bpp data is rather irrelevant nowadays. Your mileage may 
vary, of course. And in order to be consequent, this would have to be 
done for 1 bpp QuickTime Animation (qtrle) as well.

with your patch applied, this one should of course be ignored.

Mats

-- 
Mats Peterson
http://matsp888.no-ip.org/~mats/


More information about the ffmpeg-devel mailing list