[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