[FFmpeg-devel] [PATCH] add context to av_log() calls

Michael Niedermayer michaelni
Mon Feb 16 17:30:35 CET 2009


On Mon, Feb 16, 2009 at 05:27:08PM +0100, Benoit Fouet wrote:
> On 02/16/2009 05:20 PM, Michael Niedermayer wrote:
> > On Mon, Feb 16, 2009 at 05:13:15PM +0100, Benoit Fouet wrote:
> >   
> >> Hi,
> >>
> >> On 02/16/2009 05:00 PM, Michael Niedermayer wrote:
> >>     
> >>> On Mon, Feb 16, 2009 at 02:35:23PM +0100, Benoit Fouet wrote:
> >>>       
> > [...]
> >   
> >>>> Index: libavformat/avidec.c
> >>>> ===================================================================
> >>>> --- libavformat/avidec.c	(revision 17366)
> >>>> +++ libavformat/avidec.c	(working copy)
> >>>> @@ -87,7 +87,7 @@ static void print_tag(const char *str, u
> >>>>  }
> >>>>  #endif
> >>>>  
> >>>> -static int get_riff(AVIContext *avi, ByteIOContext *pb)
> >>>> +static int get_riff(AVFormatContext *s, AVIContext *avi, ByteIOContext *pb)
> >>>>     
> >>>>         
> >>> this seems redundant
> >>>
> >>>
> >>>   
> >>>       
> >> I'm not sure I understand. Is there an av_log()-friendly context already
> >> available in one of the two parameters ?
> >>     
> >
> > AVIContext *avi = s->priv_data;
> > so pasing AVIContext seems redundant
> >
> >
> > [...]
> >   
> >
> 
> of course, silly me... new patch attached.

ok

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

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090216/c1aabf64/attachment.pgp>



More information about the ffmpeg-devel mailing list