[MPlayer-dev-eng] [PATCH] mov fragments support for demux_mov

[Diego Biurrun]

On Thu, Nov 08, 2007 at 11:28:14AM +0100, David Romacho Rosell wrote:
> Diego Biurrun wrote:
> > On Mon, Oct 29, 2007 at 02:50:12PM +0100, David Romacho Rosell wrote:
> >   
> >> This is my second attempt to have this patch revised. This patch adds 
> >> support for movie fragments in mov files (as specified in the 
> >> ISO/IEC-14496). The changes only affect the libmpdemux/demux_mov.c file. 
> >
> > Can you point us at some samples for testing?
> >   
> I have a sample which is about 1.5 Mb. How can I make is available to you ?
> I have a FTP server where I could create an account just to access that 
> file (if that helps)...

Upload it to our FTP server, see the bug reporting guidelines.

> >> Regarding the size of the patch, I would like to list all the new features 
> >> that this patch adds:
> >
> > Is it possible to split this patch into small pieces?  Big patches
> > always have a hard time getting committed...
> 
> Ok, I will try to send smaller patches that just do a part of the task 
> so someone can review them. Would that be better (faster) approach ?

The smaller a patch, the faster it gets reviewed.

Diego