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

Michael Niedermayer michaelni
Wed Jan 24 23:29:44 CET 2007


Hi

On Wed, Jan 24, 2007 at 02:17:34PM +0100, Michel Bardiaux wrote:
> Michael Niedermayer wrote:
> >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
> >
> >
> 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.

looks ok


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

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- 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/2c90b627/attachment.pgp>



More information about the ffmpeg-cvslog mailing list