[FFmpeg-cvslog] r16616 - in trunk/libavformat: avc.c avc.h flvenc.c matroskaenc.c movenc.c

[Luca Abeni]

Hi Baptiste,

Baptiste Coudurier wrote:
> Hi guys,
> 
> lucabe wrote:
>> Author: lucabe
>> Date: Thu Jan 15 15:03:07 2009
>> New Revision: 16616
>>
>> Log:
>> Do not reallocate AVPacket's data when muxing a packet
>>
> 
> Argh, I just noticed a problem with the patch, this broke muxing if nal
>  startcodes are not 4 bytes :(

Ops... Sorry about that.


> I think this must be reverted and the other approach must be used
> (allocating a new buffer).
> 
> I will implement this in flv and mov muxer, I don't know much about mkv
> muxer.

I was going to check how to revert the patch using the fancy svn
features, but I see that you already committed something else to
avc.{c,h}... Should the patch still be reverted? Or should I revert only
the part not touching avc.c? Or did you already fix the problem in the
meanwhile?


			Thanks,
				Luca