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

[Michael Niedermayer]

Hi

On Wed, Jun 27, 2007 at 02:04:02AM +0200, Vitor wrote:
> 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.

looks ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

There will always be a question for which you do not know the correct awnser.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070627/1107e993/attachment.pgp>