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

[Benoit Fouet]

On 02/16/2009 05:01 PM, Michael Niedermayer wrote:
> On Mon, Feb 16, 2009 at 12:08:19PM +0100, Benoit Fouet wrote:
>   
>> 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.
>>     
>
> ok
>
> [...]
>   
>   

applied

Ben