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

[Vitor]

Hi

Michael Niedermayer wrote:
> Hi
> 
> [...]
>> Index: libavformat/yuv4mpeg.c
>> ===================================================================
>> --- libavformat/yuv4mpeg.c	(revision 9440)
>> +++ libavformat/yuv4mpeg.c	(working copy)
>> @@ -166,11 +166,6 @@
>>      return 0;
>>  }
>>  
>> -static int yuv4_write_trailer(AVFormatContext *s)
>> -{
>> -    return 0;
>> -}
>> -
>>  #ifdef CONFIG_YUV4MPEGPIPE_MUXER
>>  AVOutputFormat yuv4mpegpipe_muxer = {
>>      "yuv4mpegpipe",
>> @@ -182,7 +177,7 @@
>>      CODEC_ID_RAWVIDEO,
>>      yuv4_write_header,
>>      yuv4_write_packet,
>> -    yuv4_write_trailer,
>> +    NULL,
>>      .flags = AVFMT_RAWPICTURE,
>>  };
> 
> the NULL here can be removed too
> 

You're right. Patch attached.

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