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

Benoit Fouet benoit.fouet
Fri Feb 27 09:46:01 CET 2009


On 02/27/2009 09:18 AM, Luca Abeni wrote:
> Benoit Fouet wrote:
>   
>> On 02/16/2009 02:35 PM, Benoit Fouet wrote:
>>     
>>> Hi,
>>>
>>> here are some patches to $subj in libavformat
>>> (note: I had to add the context for avi and ape as a parameter to a
>>> function)
>>>
>>>       
>> some of the patches lack reviews:
>>     libavformat/mov.c      : Michael or Baptiste ?
>>     libavformat/txd.c      : Ivo ?
>>     libavformat/rtp_h264.c : Luca ?
>>     
> 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 ?

Ben

-------------- next part --------------
A non-text attachment was scrubbed...
Name: rtp_h264.c.diff
Type: text/x-patch
Size: 3931 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090227/f603ab5e/attachment.bin>



More information about the ffmpeg-devel mailing list