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

[Diego Biurrun]

On Wed, Jan 24, 2007 at 02:17:34PM +0100, Michel Bardiaux wrote:
> Michael Niedermayer wrote:
> >
> >On Wed, Jan 24, 2007 at 12:12:45PM +0100, Michel Bardiaux wrote:
> >>michael wrote:
> >>>
> >>>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
> >>>
> >>> --- 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
> >
> Here it is. Compiles of course (with DEBUG_SI defined), and passes 
> regression (up to the point where it says "This test requires FFmpeg to 
> be compiled with --enable-gpl") but I dont know how much that means, and 
> I dont have MPEGTS files handy for a more targetted test.

Recompile with --enable-gpl and rerun the regression tests.

Diego