[FFmpeg-devel] Patch to add support for handling multiple edit lists in MOV files

Krishna Gadepalli krishna.k.gadepalli
Fri Feb 20 18:41:16 CET 2009


Can I just rewrite the same logic from scratch if we dont want to deal with
getting the code released under LGPL ?
If not, I guess we can ask the Xine folks
-krishna

On Thu, Feb 19, 2009 at 6:48 PM, Baptiste Coudurier <
baptiste.coudurier at gmail.com> wrote:

> Hi,
>
> On 2/18/2009 1:41 PM, Krishna Gadepalli wrote:
> > I have attached a patch to add support for multiple edit list in MOV
> files.
> > I ported this over from Xine (demux_qt.c)It seems to fix the av sync
> issue
> > in menace00.mov and bedroses.mov (hard to see this one) in
> > http://samples.mplayerhq.hu/mov/editlist
> > I cant see (or rather hear) the av sync issue with the other samples in
> that
> > directory.
> >
> > This patch removes the time_offset code which addressed only the single
> > edit-list case - since it handles that case also.
> >
> > Any feedback is appreciated...
>
> Nice to see someone working on this.
>
> However, If I'm not mistaken, Xine license is GPL, while mov.c is LGPL,
> this is a problem, unless you can get author(s) approval to relicense
> this code.
>
> --
> Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
> Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
> checking for life_signs in -lkenny... no
> FFmpeg maintainer                                  http://www.ffmpeg.org
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>




More information about the ffmpeg-devel mailing list