[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