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

Benoit Fouet benoit.fouet
Mon Mar 2 10:32:00 CET 2009


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)

Ben





More information about the ffmpeg-devel mailing list