[FFmpeg-devel] [PATCH 1/2] lavf/avienc: Simplify palette handling

[Mats Peterson]

On 02/27/2016 08:44 PM, Reimar Döffinger wrote:
> On Sat, Feb 27, 2016 at 07:52:52PM +0100, Mats Peterson wrote:
>> On 02/27/2016 07:46 PM, Reimar Döffinger wrote:
>>> On Sat, Feb 27, 2016 at 07:40:44PM +0100, Mats Peterson wrote:
>>>> On 02/27/2016 07:18 PM, Reimar Döffinger wrote:
>>>>> That's not what e.g. the matroska demuxer produces, so stream
>>>>> copying with some generic remuxing example I am really sure that is
>>>>> NOT how the packets come in.
>>>>>
>>>>
>>>> Stream copying of raw palettized data seldom works well, due to different
>>>> strides of AVI (4-byte aligned), QuickTime (2-byte aligned) and nut
>>>> (unnaligned).
>>>
>>> "Seldom"? That's only an issue (for 8 bpp) if the width
>>> is not a multiple of 4.
>>
>> It's an issue for 1 bpp data as well.
>
> Not the parsing I meant :)
> For 8bpp it can only happen if the width is not a multiple of 4.
> For 1bpp it can only happen if the width is not a multiple of 16
> (which is quite a bit more likely and thus I didn't want to use that one
> as example).
> _______________________________________________

I know you wouldn't want to :) Never mind, it's a minor issue compared 
to the others, as you say yourself.

Mats