[FFmpeg-devel] [PATCH] Implement read_file() cmdutils.c option
Michael Niedermayer
michaelni
Thu Apr 1 01:22:55 CEST 2010
On Thu, Apr 01, 2010 at 01:03:32AM +0200, Stefano Sabatini wrote:
> On date Wednesday 2010-03-31 23:32:09 +0200, Michael Niedermayer encoded:
> > On Wed, Mar 31, 2010 at 10:06:33PM +0200, Stefano Sabatini wrote:
> [...]
> > > Subject: [PATCH 1/2] Implement cmdutils.c:read_file(), and use it in ffmpeg.c for reading
> > > the second pass encoding log file.
> > >
> > > ---
> > > cmdutils.c | 23 +++++++++++++++++++++++
> > > cmdutils.h | 11 +++++++++++
> > > ffmpeg.c | 21 ++++-----------------
> > > 3 files changed, 38 insertions(+), 17 deletions(-)
> > >
> > > diff --git a/cmdutils.c b/cmdutils.c
> > > index c2c44c6..8c5561c 100644
> > > --- a/cmdutils.c
> > > +++ b/cmdutils.c
> > > @@ -639,3 +639,26 @@ int read_yesno(void)
> > >
> > > return yesno;
> > > }
> > > +
> > > +int read_file(const char *filename, char **bufptr, size_t *size)
> > > +{
> > > + FILE *f = fopen(filename, "r");
> > > +
> > > + if (!f) {
> > > + fprintf(stderr, "Cannot read file '%s': %s\n", filename, strerror(errno));
> > > + return AVERROR(errno);
> > > + }
> > > + fseek(f, 0, SEEK_END);
> > > + *size = ftell(f);
> > > + fseek(f, 0, SEEK_SET);
> > > + *bufptr = av_malloc(*size + 1);
> > > + if (!*bufptr) {
> > > + fprintf(stderr, "Could not allocate file buffer\n");
> > > + return AVERROR(ENOMEM);
> > > + }
> >
> > missing fclose()
>
> Ouch, fixed, regards.
> --
> FFmpeg = Formidable Fierce Mastering Peaceful Earthshaking Game
> cmdutils.c | 24 ++++++++++++++++++++++++
> cmdutils.h | 11 +++++++++++
> ffmpeg.c | 21 ++++-----------------
> 3 files changed, 39 insertions(+), 17 deletions(-)
> c35038d6808ce696d5f1a8dcc0dd8df1b5c56ca6 0001-Implement-cmdutils.c-read_file-and-use-it-in-ffmpeg..patch
> >From bad7052f9c1635089c4eeb27d9593941fa4f7fc8 Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> Date: Sun, 28 Mar 2010 16:38:42 +0200
> Subject: [PATCH 1/2] Implement cmdutils.c:read_file(), and use it in ffmpeg.c for reading
> the second pass encoding log file.
should be ok
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- 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/20100401/6b93b107/attachment.pgp>
More information about the ffmpeg-devel
mailing list