[FFmpeg-devel] [PATCH] Make long_names in lavf/lavdev optional depending on CONFIG_SMALL

Stefano Sabatini stefano.sabatini-lala
Tue Jun 3 14:44:13 CEST 2008


On date Tuesday 2008-06-03 13:50:20 +0200, Diego Biurrun encoded:
> On Tue, Jun 03, 2008 at 12:04:27PM +0200, Stefano Sabatini wrote:
> > 
> > I missed bktr in the previous one.
> > 
> > --- libavformat/avformat.h	(revision 13644)
> > +++ libavformat/avformat.h	(working copy)
> > @@ -185,6 +185,10 @@
> >  
> > +     * descriptive name for the format, meant to be more human readable than \p name
> 
> Capitalize, add period and please break this overly long line.
> 
> > @@ -216,6 +220,10 @@
> >  
> > +     * descriptive name for the format, meant to be more human readable than \p name
> 
> ditto

mmh... OK, I really used the same description used for
AVCodec.long_name, but I see that this way is more consistent with the
rest of the file...

> > --- libavformat/http.c	(revision 13644)
> > +++ libavformat/http.c	(working copy)
> > @@ -134,6 +134,7 @@
> >      return ret;
> >  }
> > +
> >  static int http_getc(HTTPContext *s)
> 
> I guess this part is accidental.

Oops.

Regards.
-- 
FFmpeg = Fucking and Fast MultiPurpose ExchanGer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: conditional-lavf-long-names-02.patch
Type: text/x-diff
Size: 52377 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080603/4487cd1b/attachment.patch>



More information about the ffmpeg-devel mailing list