[FFmpeg-devel] [PATCH]pes packetizer

Xiaohui Sun sunxiaohui
Fri Aug 31 13:32:54 CEST 2007


Diego Biurrun wrote:
> On Fri, Aug 31, 2007 at 07:09:41PM +0800, Xiaohui Sun wrote:
>   
>> Diego Biurrun wrote:
>>     
>>> On Fri, Aug 31, 2007 at 05:43:35PM +0800, Xiaohui Sun wrote:
>>>   
>>>       
>>>> [...]
>>>>         
>>> The main complaint was that you should split your patch into several
>>> more manageable and separate parts.  But you send in another 66kB beast.
>>> Either I am missing something (well possible, I didn't check in detail)
>>> or you should start sending in small pieces.
>>>       
>> split a file into two files will cause the patch approximately two times 
>> the original files (mpegenc.c is 40k)
>> should I put the modification of each file(mpegenc.c mpeg_pes_enc.c 
>> Makefile mpeg_pes.h) into different patches, but that
>> seems meaningless to each patch and could not be compiled :-(
>>     
>
> I'll try to say things more clearly: If the function renaming and the
> cosmetics can be applied first, then you should send these in
> separately.
>
>   
what about first send/apply the main patch(split) then send the 
cosmetics and renamings,
and that would be more readable IMHO

> Diego
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>
>   





More information about the ffmpeg-devel mailing list