[FFmpeg-devel] Patch to add support for handling multiple edit lists in MOV files
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...
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
> On Thu, Feb 19, 2009 at 6:48 PM, Baptiste Coudurier <
> baptiste.coudurier at gmail.com> wrote:
>> On 2/18/2009 1:41 PM, Krishna Gadepalli wrote:
>> > I have attached a patch to add support for multiple edit list in MOV
>> > I ported this over from Xine (demux_qt.c)It seems to fix the av sync
>> > 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
>> > 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
More information about the ffmpeg-devel