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

David Romacho Rosell romaxin2002 at yahoo.com.au
Thu Nov 8 13:16:47 CET 2007


Diego Biurrun wrote:
> 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
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
>
>   


The file has been uploaded to the ftp as "au30sec.mp4" and a text file 
"au30sec.txt" with a short description of its content. Once I have some 
time I will split the patch in several smaller patches.

Thanks,

  David




More information about the MPlayer-dev-eng mailing list