On Fri, 2 Sep 2011 10:47:45 -0700, Alex Converse <[email protected]> wrote: > On Thu, Sep 1, 2011 at 2:03 AM, Anton Khirnov <[email protected]> wrote: > > > > This is the first step to removing the globals plague from avtools. > > --- > > avconv.c | 4 ++-- > > avplay.c | 4 ++-- > > avprobe.c | 4 ++-- > > avserver.c | 2 +- > > cmdutils.c | 14 +++++++++----- > > cmdutils.h | 10 ++++++++-- > > ffmpeg.c | 4 ++-- > > 7 files changed, 26 insertions(+), 16 deletions(-) > > > > diff --git a/avconv.c b/avconv.c > > index 1c34c5f..2d3f3f4 100644 > > --- a/avconv.c > > +++ b/avconv.c > > @@ -3403,7 +3403,7 @@ static int read_avserver_streams(AVFormatContext *s, > > const char *filename) > > return 0; > > } > > > > -static void opt_output_file(const char *filename) > > +static void opt_output_file(void *obj, const char *filename) > > { > > AVFormatContext *oc; > > int i, err; > > Perhaps we can come up with a better name than "obj" for the options > context. octx? optctx?
Sure i don't mind, but from the POV of cmdutils it's just some opaque object. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
