[MPlayer-dev-eng] [PATCH] Movie fragment support for MOV/MP4 demuxer
David Romacho Rosell
romaxin2002 at yahoo.com.au
Mon Sep 17 17:33:10 CEST 2007
En/na Reimar Döffinger ha escrit:
> Hello,
> On Sat, Sep 15, 2007 at 06:38:26PM +0200, David Romacho Rosell wrote:
>
>> Thanks for your comments. This is the first time I send a patch and, as
>> I can see, I will need to spend some more time on the code.
>>
>
> And please do not top-post when replying.
> I do not know how you created the patch, but if you need help with something it might be
> better to send a patch against demux_mov.c before I split lschunks, that
> should already make the changes a bit easier to follow.
> I should (hopefully) also be reasonably easy to bring that patch then
> up-to-date with the merging features of e.g. svn or git (though svn is
> not that great at it and git is a bit complex to use in that regard).
>
> Greetings,
> Reimar Döffinger
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
>
>
Thanks for your comments. I am trying to get used to your rules.
I revised the patch so it does not introduce as many changes as the
previous one. I also cleaned up the cosmetics. Note that now we have a
lschunks_inmfra, lschunks_intraf, lschunks_inmoof, lschunks_inmvex
functions, each one processes box/atom needed by the movie fragments.
This code could be in lschunks but I think that in that way lschunks
does not get a mess.
Greetings,
David
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: patch_mov_traf_v2.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20070917/30619c4a/attachment.txt>
More information about the MPlayer-dev-eng
mailing list