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

[Michael Niedermayer]

On Mon, Feb 01, 2016 at 05:52:02PM +0100, Mats Peterson wrote:
> On 02/01/2016 02:58 PM, Michael Niedermayer wrote:
> >its not ok,
> >if only 2bpp are used and its possible to store just 2bpp then thats
> >what should be done by default. Same for 4bpp
> >that could be done with PAL8 and using its palette or using
> >some *_bits_per_pixel variable or a seperate PAL4/PAL2
> >many possibilities and likely even more but the end result should be
> >efficient storage so that encoders/muxers supporting 2/4 bpp should
> >use these when possible and by default without the user needing to
> >know all the details
> >
> 
> It all sounds wonderful, but it would take a lot of rewriting. And
> with the current situation, we have to use pal8 for both 1, 2, and 4
> bpp.

The problem is not using pal8 the problem is that it currently doesnt
work well at all if pal8 is used

making pal8 work well may or may not be alot of work, i dont know
but if you dont want to make pal8 work well then you also shouldnt
push that hard to use it

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160201/d841ad31/attachment.sig>