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

Benoit Fouet benoit.fouet
Mon Feb 16 17:33:04 CET 2009


On 02/16/2009 05:30 PM, Michael Niedermayer wrote:
> 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
>
> [...]
>   
>   

applied





More information about the ffmpeg-devel mailing list