On Wed, 20 Apr 2011, Diego Biurrun wrote:
> On Wed, Apr 20, 2011 at 04:24:45PM +0300, Martin Storsjö wrote:
> >
> > --- a/cmdutils.c
> > +++ b/cmdutils.c
> > @@ -155,6 +155,68 @@ static const OptionDef* find_option(const OptionDef
> > *po, const char *name){
> >
> > +/**
> > + * Prepare command line arguments for executable.
> > + * For Windows - perform wide-char to UTF-8 conversion.
> > + * Input arguments should be main() function arguments.
> > + * @param argc_ptr Arguments number (including executable)
> > + * @param argv_ptr Arguments list.
> > + */
> > +static void prepare_app_arguments(int *argc_ptr, char ***argv_ptr);
>
> How is that static forward declaration not clashing with the
> non-static ones below?
>
> Also, forward declarations are ugly and should be avoided.
Good point. The forward declaration could be skipped altogether here, and
just marking the actual functions static. Updated locally.
// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
