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

Krishna Gadepalli krishna.k.gadepalli
Fri Feb 20 18:43:42 CET 2009


ah! I guess my question was answered before I sent the mail :-)I will send
out an updated patch...
-krishna

On Fri, Feb 20, 2009 at 9:41 AM, Krishna Gadepalli <
krishna.k.gadepalli at gmail.com> wrote:

> 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