[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