[FFmpeg-devel] [PATCH] url_fskip: Warn if the url_fseek call failed
Michael Niedermayer
michaelni
Fri Jul 16 19:49:09 CEST 2010
On Fri, Jul 16, 2010 at 08:26:46PM +0300, Martin Storsj? wrote:
> On Fri, 16 Jul 2010, Michael Niedermayer wrote:
>
> > On Fri, Jul 16, 2010 at 10:19:42AM -0400, Ronald S. Bultje wrote:
> > > Hi,
> > >
> > > On Fri, Jul 16, 2010 at 9:31 AM, Martin Storsj? <martin at martin.st> wrote:
> > > > $subj - url_fskip returns void at the moment, so if the wrapped url_fseek
> > > > call fails, at least tell the poor user that is debugging it what's going
> > > > wrong.
> > >
> > > Sometimes the seek intentionally is wrong, e.g. if the demuxer tests
> > > if the protocol can seek to a particular position.
> > >
> > > > Or would it be ok to change url_fskip to return int and pass the error
> > > > code through, and check for the errors in the individual demuxers?
> > >
> > > Probably... Michael can make a call here, he's maintainer.
> >
> > passing the error code through seems more correct
>
> Ok, what about this one?
>
> I guess changing the return value from void to int doesn't break ABI on
> any platform we care about?
>
> // Martin
> avio.h | 3 ++-
> aviobuf.c | 5 +++--
> 2 files changed, 5 insertions(+), 3 deletions(-)
> c13efc44e09dfb429f8e58c1a0b87d7e84a474b7 0001-url_fskip-Return-an-error-code-if-the-url_fseek-fail.patch
> From 3afb4b2f8557600c4474149d3320300cb0e3266c Mon Sep 17 00:00:00 2001
> From: Martin Storsjo <martin at martin.st>
> Date: Fri, 16 Jul 2010 16:28:54 +0300
> Subject: [PATCH] url_fskip: Return an error code if the url_fseek failed
sould be ok
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Into a blind darkness they enter who follow after the Ignorance,
they as if into a greater darkness enter who devote themselves
to the Knowledge alone. -- Isha Upanishad
-------------- 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/20100716/dadc7d3d/attachment.pgp>
More information about the ffmpeg-devel
mailing list