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

Michel Bardiaux mbardiaux
Wed Jan 24 14:17:34 CET 2007


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.

HaND,
-- 
Michel Bardiaux
R&D Director
T +32 [0] 2 790 29 41
F +32 [0] 2 790 29 02
E mailto:mbardiaux at mediaxim.be

Mediaxim NV/SA
Vorstlaan 191 Boulevard du Souverain
Brussel 1160 Bruxelles
http://www.mediaxim.com/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: lavf_mpegts_1.pat
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20070124/c7a13889/attachment.txt>



More information about the ffmpeg-cvslog mailing list