[FFmpeg-devel] [PATCH 1/2] Make av_strerror() print the string of "Error number %d occurred" if strerror_r() did not succeed, for example because the error code is not known.

Stefano Sabatini stefano.sabatini-lala
Mon May 3 23:56:37 CEST 2010


On date Saturday 2010-05-01 20:05:22 +0200, Michael Niedermayer encoded:
> On Sat, May 01, 2010 at 01:53:24PM +0200, Stefano Sabatini wrote:
> > On date Saturday 2010-05-01 10:54:27 +0200, Michael Niedermayer encoded:
> > > On Sat, May 01, 2010 at 10:28:33AM +0200, Stefano Sabatini wrote:
> > > > This avoids the need for the application to fill the string in case
> > > > strerror_r() fails to do it for whatever reason.
> > > 
> > > this returns -1 (failure) with your patch
> > 
> > Yes, fixed.
> > 
> > Regards.
> > -- 
> > FFmpeg = Faithful & Frenzy Meaningless Peaceful Ecletic Gorilla
> 
> >  error.c |    4 ++--
> >  error.h |    3 ++-
> >  2 files changed, 4 insertions(+), 3 deletions(-)
> > 7cd2207067a327c1ac19d6359de3a543413bd276  0001-Make-av_strerror-print-the-string-of-Error-number-d-.patch
> > >From 9563de6f95c299c5d4269f8d98260acac8fe9ed9 Mon Sep 17 00:00:00 2001
> > From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> > Date: Sat, 1 May 2010 10:12:48 +0200
> > Subject: [PATCH 1/2] Make av_strerror() print the string of "Error number %d occurred" if
> >  strerror_r() did not succeed, for example because the error code is
> >  not known.
> > 
> > This avoids the need for the application to fill the string in case
> > strerror_r() fails to do it for whatever reason.
> > ---
> >  libavutil/error.c |    4 ++--
> >  libavutil/error.h |    3 ++-
> >  2 files changed, 4 insertions(+), 3 deletions(-)
> 
> looks ok if tested

Applied.
-- 
FFmpeg = Fantastic and Friendly Magnificient Patchable Evangelical Gnome



More information about the ffmpeg-devel mailing list