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

Benoit Fouet benoit.fouet
Fri Feb 27 11:31:38 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.

no, thank you.

>  I'd say that if noone complains you can apply
> it in two or three days.
>
>   

I'll do that.
the point is, no one is listed as maintainer of that file right now, I
guess it's Luca B. who should be, right ?

Ben





More information about the ffmpeg-devel mailing list