[FFmpeg-devel] [PATCH v3] lavc/rawdec: Use 16-byte line alignment for B1W0 and B0W1 video in nut

Michael Niedermayer michael at niedermayer.cc
Fri Feb 5 15:43:51 CET 2016


On Fri, Feb 05, 2016 at 03:29:36AM +0100, Mats Peterson wrote:
> On 02/05/2016 03:04 AM, Mats Peterson wrote:
> >Yes, it could (regarding stride being zero). Here's an updated patch.
> 
> 
> If you think it's extremely important to use monow for the case of
> raw 1 bpp AVI without a palette, I can remove the "Temporary
> solution" comments. But remember this will only be for AVI.

patch applied, but i think theres more that needs fixing than pal8
in nut.
Most important is that the code which chooses the pixel format for
encoding end up choosing monowhite if the inut is PAL8 with 2 matching
colors.
Then decoders could output PAL8 but encoders would receve monow if
they support it and no changes would be needed per codec

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

Rewriting code that is poorly written but fully understood is good.
Rewriting code that one doesnt understand is a sign that one is less smart
then the original author, trying to rewrite it will not make it better.
-------------- 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/20160205/ef8df413/attachment.sig>


More information about the ffmpeg-devel mailing list