[FFmpeg-devel] [PATCH] Document opt.h:av_set_string functions
Wed Jul 16 18:42:18 CEST 2008
On Tue, Jul 15, 2008 at 11:10:14PM +0200, Stefano Sabatini wrote:
> On date Wednesday 2008-07-09 01:11:27 +0200, Michael Niedermayer encoded:
> > On Tue, Jul 08, 2008 at 09:47:47PM +0200, Stefano Sabatini wrote:
> > > On date Sunday 2008-07-06 11:00:34 +0200, Stefano Sabatini encoded:
> > > > Hi all, as in subject.
> > > > --
> > > > FFmpeg = Funny and Fantastic Multimedia Portable Educated Game
> > >
> > [...]
> > > Please can you comment on this? Or I'll commit it on Saturday.
> > Would you accept a comment like
> > all your doxy improvments are rejected?
> > It costs me more time fixing them up than writing them from scratch
> Michael, I'll try to avoid to write doxy at least not related to my
> own code (yes it's getting more and more a boring time waste), only
> please let me end up just the doxyfication of this file which is in my
> todo list since too much time and I don't want to waste all the effort
> and comments spent since then.
> New try attached.
> Thanks for your patience, regards.
> FFmpeg = Funny & Frenzy Multimedia Powered Experimenting Guru
> Index: libavcodec/opt.h
> --- libavcodec/opt.h (revision 14247)
> +++ libavcodec/opt.h (working copy)
> @@ -99,10 +99,28 @@
> const AVOption *av_find_opt(void *obj, const char *name, const char *unit, int mask, int flags);
> + * @see av_set_string2()
> + */
> attribute_deprecated const AVOption *av_set_string(void *obj, const char *name, const char *val);
> - * Sets the field of obj with the given name to value.
> + * Sets a field of \p obj with the value in \p val.
> + *
> + * @param[in] obj a pointer to a struct whose first element is a
> + * pointer to an #AVClass
a struct whose first element is a pointer to an #AVClass
> + * @param[in] name the name of the option contained in \p obj
> + * corresponding to the field to set.
the name of the field
> + * @param[in] val A string containing the value to parse and set
> + * in \p obj.
the value. :)
If you want, you could say in a seperate sentance that
If the field is not of a string type, then the given string is parsed.
> + * If the option is of numeric type, it has to be a numeric scalar
> + * optionally with a SI postfix or a named scalar.
"SI postfixes and some named scalars are supported"
> Behavior with more
> + * than one scalar and +- infix operators is undefined.
> + * If the option is of type flags, it has to be a numeric scalar or a
> + * named flag.
i agree with the content but not with how its formulated
> Prefixing a flag with + causes it to be set without
> + * affecting the other flags,
> - similarly unsets a flag.
similarly '-' unsets a flag.
> + * @return a pointer to the option set
> or NULL if no option has been
> + * found \e or the value \p val is not valid
or NULL if no matching AVOption exists, ...
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
There will always be a question for which you do not know the correct awnser.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel