[FFmpeg-devel] ffmepg.c: change "fprintf(stderr" to "av_log(NULL, AV_LOG_ERROR"

Benoit Fouet benoit.fouet
Mon Feb 16 12:08:19 CET 2009


On 02/16/2009 11:38 AM, Benoit Fouet wrote:
> Hi,
>
> On 02/16/2009 11:35 AM, Pawe? Ko?odziej wrote:
>   
>> Hi,
>> In ffmpeg.c there is a lot (about 80) places like:
>>
>>   fprintf(stderr, "yada yada yada\n");
>>   av_exit(1);
>>
>>   
>>     
>
> these are error messages from ffmpeg application, and should IMHO remain
> as such.
>
>   
>> Is there any reason to not change "fprintf(stderr" to
>> "av_log(NULL,AV_LOG_ERROR" ? If not then I will do this change. 
>>
>>   
>>     
>
> av_log is used inside libav* to log things, ffmpeg is not part of libav*
> but uses them...
>
>   

as a result, here is a patch to change calls to av_log to fprintf to stderr.

Ben

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



More information about the ffmpeg-devel mailing list