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

[Michael Niedermayer]

On Sat, Apr 11, 2009 at 06:37:10PM +0200, Kostya Shishkov wrote:
> On Sat, Apr 11, 2009 at 6:23 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Sat, Apr 11, 2009 at 06:04:07PM +0200, Kostya Shishkov wrote:
> >> On Sat, Apr 11, 2009 at 3:51 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> >> > On Sat, Apr 11, 2009 at 03:36:39PM +0200, Reimar D?ffinger wrote:
> >> [...]
> >> >> Also I think it would already be quite simple to fix if we just allowed
> >> >> packets to contain only palette (without any PKT_FLAG_PALETTE), since
> >> >> allocating a buffer and merging the palette stuff together with a video
> >> >> buffer is just a pain to implement. The problem though is that no
> >> >> keyframes at all would exist then.
> >> >
> >> > i think palette belongs in the packet with a frame not in a seperate packet
> >> > a palette is like a quantization tables in mpeg, we should treat it like
> >> > one
> >>
> >> Think about palettes in AVI - they are rather codec-agnostic. I think
> >> it's better to pass palettes as separate packets that can be handled
> >
> > that requires support for zero duration packets and requires code to
> > merge these packets with frames in muxers that are not called avi
> > so unless someone implements these, its not an option because it wouldnt
> > work with remuxing
> 
> zero-duration packets - yes and it would be an unpleasant hack

> as for remuxing - why should we bother? it's container-specific

if we bother fixing AVPalette it should be done correctly, not in a
way that wont work with remuxing


> feature for AVI so remuxed files would be probably decoded only by
> FFmpeg

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When the tyrant has disposed of foreign enemies by conquest or treaty, and
there is nothing more to fear from them, then he is always stirring up
some war or other, in order that the people may require a leader. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090411/5aacedc2/attachment.pgp>