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

[Michael Niedermayer]

On Sun, Jan 31, 2016 at 10:08:41PM +0100, Mats Peterson wrote:
> On 01/31/2016 03:07 PM, Michael Niedermayer wrote:
> >On Sun, Jan 31, 2016 at 01:27:22PM +0100, Mats Peterson wrote:
> >>I don't really appreciate that you're doing things behind my back,
> >>Michael. There's nothing mentioned on the ffmpeg-devel mailing lis
> >
> >indeed, i should have clearly stated that i applied the pal8
> >patches with the plan to use pal8 only when neccessary.
> >I had asked you to implement exactly that but it seemed you didnt
> >know how to do that cleanly and simply so i thought "no problem,
> >i know how to do that, ill do it"
> >i had not expected this to be controversal
> >
> >
> 
> Even if you stubbornly decide to keep your patch, it is incorrect
> anyway. If a 1 bpp AVI contains a palette, black & white or not,

If a AVI contains a palette one might argue its a paletted avi
either way i dont know, i dont have such a file
it would be needed to have real world files in that category
to know if treating them as pal8 is better or not


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

it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
-------------- 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/20160131/6851d162/attachment.sig>