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

Benoit Fouet benoit.fouet
Tue Mar 3 17:19:54 CET 2009


On 03/02/2009 10:32 AM, Benoit Fouet wrote:
> On 02/27/2009 11:26 AM, Luca Abeni wrote:
>   
>> Benoit Fouet wrote:
>> [...]
>>   
>>     
>>>> I am not sure if I am the right Luca, anyway I had a look at the patch.
>>>> I suspect the
>>>> -            av_log(NULL, AV_LOG_DEBUG,
>>>> +            av_log(stream, AV_LOG_DEBUG,
>>>> kind of changes are wrong, because stream here is an AVStream and I think
>>>> AVStream does not contain an AVClass... So, I don't think it can be used
>>>> as a context for av_log().
>>>>
>>>>       
>>>>         
>>> indeed, how does the attached one look like ?
>>>     
>>>       
>> This looks better, thanks. I'd say that if noone complains you can apply
>> it in two or three days.
>>
>>   
>>     
>
> applied.
>
> I think I'll apply the last one (i.e. the one to txd.c) this week,
> unless someone speaks up (it is rather trivial after all)
>
>   

applied

Ben




More information about the ffmpeg-devel mailing list