[Ffmpeg-cvslog] r7674 - trunk/libavformat/mpegts.c

Michael Niedermayer michaelni
Wed Jan 24 13:37:30 CET 2007


Hi

On Wed, Jan 24, 2007 at 12:12:45PM +0100, Michel Bardiaux wrote:
> michael wrote:
> >Author: michael
> >Date: Tue Jan 23 22:41:30 2007
> >New Revision: 7674
> >
> >Modified:
> >   trunk/libavformat/mpegts.c
> >
> >Log:
> >"converts printf's to avlog's" patch by Daniel Kristjansson / danielk 
> >mrl.nyu edu
> >another 1 year old patch forgoten by mans
> >trailing whitespace removial by me
> >
> >
> >Modified: trunk/libavformat/mpegts.c
> >==============================================================================
> >--- trunk/libavformat/mpegts.c	(original)
> >+++ trunk/libavformat/mpegts.c	Tue Jan 23 22:41:30 2007
> >@@ -162,7 +162,7 @@
> >     MpegTSSectionFilter *sec;
> > 
> > #ifdef DEBUG_SI
> >-    printf("Filter: pid=0x%x\n", pid);
> >+    av_log(NULL, AV_LOG_DEBUG, "Filter: pid=0x%x\n", pid);
> > #endif
> [snip]
> 
> Why NULL? Could be ts->stream, or add an AVClass at the top of 
> MpegTSContext. (I dont propose a patch because it would depend on the 
> choice made)

it should be AVFormatContext of course instead of NULL, patch welcome

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

If you really think that XML is the answer, then you definitly missunderstood
the question -- Attila Kinali
-------------- 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-cvslog/attachments/20070124/5a863d92/attachment.pgp>



More information about the ffmpeg-cvslog mailing list