[FFmpeg-devel] [PATCH] IFF: Add grayscale support to decoder

[Ronald S. Bultje]

Hi,

On Mon, May 10, 2010 at 1:25 PM, Sebastian Vater
<cdgs.basty at googlemail.com> wrote:
> Ronald S. Bultje a ?crit :
>> But can it get < 0? And should it get < 0? And if so, what does that mean?
>
> It means that there's no actual palette data itself attached from an old
> demuxer, the new decoder will then subtract the custom extra data size
> and this will < 0, of course this should be handled as no palette and
> therefore use grayscale palette ;)

This sounds buggy to me. How many bytes are in extradata if count < 0?
count should never be smaller than size / 3, suggesting that count < 0
is in fact a bug.

Ronald