[FFmpeg-devel] maintainer duties (was: Re: [PATCH] fix speex sample)

[Baptiste Coudurier]

On Sat, Apr 11, 2009 at 09:23:49AM -0400, Ronald S. Bultje wrote:
> Hi,
> 
> On Sat, Apr 11, 2009 at 7:49 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > It has been decided to put the palette in the AVPackets
> > how to do this is a question per decoder and per demuxer, some may have a
> > natural way to pass the palette, some might even say in a spec how to do it
> > (though i guess that would be the exception if it applies to any at all)
> > the rest could just have a 'P'<len> vs. 'D' prefixed to packets
> 
> I think I said this before, I don't like this (conceptually). Can't we
> just add a PKT_FLAG_PALETTE? It's exactly the same as a prefix, except
> that it does not actually touch the bitstream. Now that Thilo added
> all these AVPackets to the decode() functions, let's use them.
> 

I agree with the principle. I'm more in favor of
AVPacket->palette_data field.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA    
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
checking for life_signs in -lkenny... no
FFmpeg maintainer                                  http://www.ffmpeg.org