[FFmpeg-devel] A question about AVOutputFormat.write_trailer (cosmetical)

Vitor vitor1001
Tue Jun 26 22:48:21 CEST 2007


Hi

Michael Niedermayer wrote:
> On Tue, Jun 26, 2007 at 09:07:50PM +0200, Vitor wrote:
>> Hi,
>>
>> For a good number of muxers (notably in raw.c), there is an empty 
>> function that is passed to AVOutputFormat.write_trailer. The only place 
>> it is called checks for a NULL pointer, and almost every muxer pass a 
>> NULL pointer to AVOutputFormat.interleave_packet. So would it be ok to 
>> send a patch to remove all those empty functions and set write_trailer 
>> to NULL instead?
> 
> yes

Patch attached.

-Vitor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: trailer.diff
Type: text/x-patch
Size: 4832 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070626/ee61e60a/attachment.bin>



More information about the ffmpeg-devel mailing list