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

Mats Peterson matsp888 at yahoo.com
Sun Jan 31 13:34:26 CET 2016


On 01/31/2016 01:33 PM, wm4 wrote:
> On Sun, 31 Jan 2016 13:27:22 +0100
> Mats Peterson <matsp888-at-yahoo.com at ffmpeg.org> wrote:
>
>> I don't really appreciate that you're doing things behind my back,
>> Michael. There's nothing mentioned on the ffmpeg-devel mailing list
>> about your "monowhite switching patch". Furthermore, to me it's highly
>> unncecessary to use monowhite whatsoever. The space savings are quite
>> irrelevant nowadays, and the logic to detect whether a file is black &
>> white only creates more noise in the code. Here's a patch that restores
>> the old behaviour of only using pal8 for 1 bpp video, and removes
>> superfluous monowhite stuff.
>
> Less complexity for such a corner case sounds indeed better

Yes, and what a corner case it is...

Mats



More information about the ffmpeg-devel mailing list