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

[Michael Niedermayer]

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

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090216/861349be/attachment.pgp>