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

Michael Niedermayer michael at niedermayer.cc
Mon Feb 1 16:04:15 CET 2016


On Mon, Feb 01, 2016 at 08:17:54AM +0100, Mats Peterson wrote:
> On 02/01/2016 08:02 AM, Mats Peterson wrote:
> >Should be "1 bpp video in AVI", not just "1 bpp video".
> >
> 
> Your patch that switches to monow when a 1 bpp AVI doesn't contain a
> palette is rather kludgy and redundant to me. And users will undoubtly
> be surprised by the use of monow for the unique case of 1 bpp AVI
> without a palette, rather than using pal8 consistently for all 1 bpp AVI
> files, just like it is done for 2 and 4 bpp. It is also semantically
> incorrect to use monow for 1 bpp AVI whatsoever, since there is no such
> thing as "monochrome" video data in AVI.
> 
> I don't understand what's so important about retaining monow inside
> FFmpeg when it comes to AVI, when it's obviously OK to convert 2 and 4
> bpp to pal8. Once again, if you want a straight passthrough of 1 bpp AVI
> video data, just use "-c:v copy".
> 
> Judging by the lack of objections against this patch from people
> (except you, Michael) on the mailing list, I take it as a "silent
> approval".

fix the regressions your patch causes and iam happy to apply it

2-8 times output filesize explosion with default parameters being
the main regression


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 3
"Rare item" - "Common item with rare defect or maybe just a lie"
"Professional" - "'Toy' made in china, not functional except as doorstop"
"Experts will know" - "The seller hopes you are not an expert"
-------------- 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/7a88ee68/attachment.sig>


More information about the ffmpeg-devel mailing list