[FFmpeg-devel] [PATCH v9] lavc/rawdec: Remove monowhite switching code for 1 bpp AVI without a palette

[Mats Peterson]

On 02/12/2016 01:09 PM, Mats Peterson wrote:
> On 02/12/2016 01:03 PM, Mats Peterson wrote:
>> Well, it's not because of my patch. It was the same before it. And you
>> have more knowledge than me when it comes to fixing the muxing code. I
>> have mostly dealt with the demuxing side so far, since that's where I
>> feel decently comfortable.
>>
>> Mats
>
> I admit it sounded like it was because of my patch, but I worded it in a
> confusing way, I guess. I meant that since my patch will make 1 bpp raw
> AVI files use pal8 internally, to produce a 1 bpp file, you can either
> use "-vcodec copy", or "-vcodec rawvideo -pix_fmt mono", and the latter
> one will produce files with incorrect strides. It has nothing to do with
> my patch per se.
>

To be exact, AVI and QuickTime files produced with "-vcodec rawvideo", 
regardless of bit depth, will use the minimum possible stride like nut, 
which is not according to specs. AVI should use 4-byte aligned strides, 
and QuickTime 2-byte aligned strides.

Mats