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

[Mats Peterson]

On 02/12/2016 01:41 PM, Mats Peterson wrote:
> 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.
>

And this is of course not considering the weird way of storing the 
palette in the frames for 8 bpp raw AVI and QuickTime, which clearly is 
not according to specs. For nut it seems sensible, though.

Mats