[FFmpeg-devel] [PATCH] Move AVOptions to specific options.c files in lavc and lavf

[Stefano Sabatini]

On date Thursday 2009-01-08 02:05:35 +0100, Michael Niedermayer encoded:
> On Thu, Jan 08, 2009 at 01:32:04AM +0100, Stefano Sabatini wrote:
> > Hi all,
> > 
> > the libavcodec/options.c file has been generated with svn copy, the
> > libavformat/options.c was not IMHO worth of it since it is few lines.
> > 
> > This should help to reduce the clutter in the corresponding utils.c
> > files.
> 
> 
> [...]
> > Index: libavcodec/options.c
> > ===================================================================
> > --- libavcodec/options.c	(revision 16492)
> > +++ libavcodec/options.c	(working copy)
> > @@ -1,8 +1,4 @@
> >  /*
> > - * utils for libavcodec
> > - * Copyright (c) 2001 Fabrice Bellard.
> > - * Copyright (c) 2002-2004 Michael Niedermayer <michaelni at gmx.at>
> > - *
> >   * This file is part of FFmpeg.
> >   *
> >   * FFmpeg is free software; you can redistribute it and/or
> > @@ -21,368 +17,14 @@
> [...]
> > Index: libavformat/options.c
> > ===================================================================
> > --- libavformat/options.c	(revision 0)
> > +++ libavformat/options.c	(revision 0)
> > @@ -0,0 +1,83 @@
> > +/*
> > + * This file is part of FFmpeg.
> > + *
> > + * FFmpeg is free software; you can redistribute it and/or
> > + * modify it under the terms of the GNU Lesser General Public
> > + * License as published by the Free Software Foundation; either
> > + * version 2.1 of the License, or (at your option) any later version.
> > + *
> > + * FFmpeg is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> > + * Lesser General Public License for more details.
> > + *
> > + * You should have received a copy of the GNU Lesser General Public
> > + * License along with FFmpeg; if not, write to the Free Software
> > + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
> > + */
> 
> your patches look strangely inconsistent

Do you mean that I should use svn copy even for libavformat too?
 
> [...]
> > Property changes on: libavformat/options.c
> > ___________________________________________________________________
> > Added: svn:eol-style
> >    + native
> > 
> 
> hmm

Ops, that's the fault of my .subversion/config file.

Regards.
-- 
FFmpeg = Frightening and Friendly Minimalistic Portable Enhancing Glue