[FFmpeg-devel] [PATCH] Add libavsequencer.

[Michael Niedermayer]

On Thu, Aug 19, 2010 at 10:12:31PM +0200, Sebastian Vater wrote:
> Vitor Sessak a ?crit :
> > On 08/19/2010 07:48 PM, Sebastian Vater wrote:
> >> Ronald S. Bultje a ?crit :
> >>> Hi,
> >>>
> >>> On Thu, Aug 19, 2010 at 10:21 AM, Stefano Sabatini
> >>> <stefano.sabatini-lala at poste.it>  wrote:
> >>> [..]
> >>>
> >>>> The other advantage of such approach is that it would be simpler to
> >>>> keep all the new code disabled by default.
> >>>>
> >>>
> >>>
> >>
> >> Hey Ronald&&  Stefano!
> >>
> >>> We could consider enabling the code by default, if:
> >>> - it's not too big
> >>> - it works
> >>> - has been properly reviewed
> >>> I think it's doable. :-).
> >>>
> >>
> >> Yes it is, but is still some way until we arrive at that point. ;-)
> >> But even if it would work perfectly right now, there's another point to
> >> keep it in avseq now, since IFF-TCM1 is, apart from us here and mine,
> >> practically unused, therefore an IFF-TCM1 demuxer/decoder is only usable
> >> for us right, now.
> >>
> >> There's no reason to make all external deps on FFmpeg making some
> >> hundred KB's larger for a feature which practically nobody can use now.
> >
> > After getting TCM decoding working, is adding MOD support is trivial?
> > If yes, it all should turn into a popular feature very soon, so this
> > is not a big deal.
> 
> Yes, basically it's just a port from my already 68k asm MOD support
> external lib using TuComposer. The same applies to S3M, XM, IT and
> FC13/14. Just MED will be new.
> >
> >
> >> So to summarize, let's keep us for now, avseq as a lib as it already is.
> >
> > I don't really understand which arguments in your email supports this
> > idea.
> >
> 
> Because practically nobody actually can use this codec currently, so
> having it default disabled and all related stuff is a better idea, I think.
> 
> It's clearer what should not be used right now, if all is in dir
> libavsequencer/* instead of having that in libavcodec. In lavc we would
> have the header files spread all around during 100 of other codecs.
> 
> I'ld prefer the way to keep the lavseq stuff as it for now and later
> decide when all works if we move that from lavseq to lavcore or sth. else.

i also think lavseq should stay in lavseq for now

[...]

-- 
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: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100820/4391ffb5/attachment.pgp>